[FFmpeg-devel] [PATCH] avfilter/vaf_spectrumsynth: Move "break" up

[Michael Niedermayer]

Fixes CID1351347

Signed-off-by: Michael Niedermayer 
---
 libavfilter/vaf_spectrumsynth.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavfilter/vaf_spectrumsynth.c b/libavfilter/vaf_spectrumsynth.c
index ab9a69b..ba14d8d 100644
--- a/libavfilter/vaf_spectrumsynth.c
+++ b/libavfilter/vaf_spectrumsynth.c
@@ -442,11 +442,11 @@ static int try_push_frames(AVFilterContext *ctx)
 case SCROLL:
 s->xpos = s->xend - 1;
 ret = try_push_frame(ctx, s->xpos);
+break;
 case RSCROLL:
 s->xpos = 0;
 ret = try_push_frame(ctx, s->xpos);
 break;
-break;
 case FULLFRAME:
 for (x = 0; x < s->xend; x++) {
 ret = try_push_frame(ctx, x);
-- 
1.7.9.5

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter/vaf_spectrumsynth: Move "break" up

[Michael Niedermayer]

On Sun, Feb 07, 2016 at 10:37:20PM +0100, Paul B Mahol wrote:
> On 2/7/16, Michael Niedermayer  wrote:
> > Fixes CID1351347
> >
> > Signed-off-by: Michael Niedermayer 
> > ---
> >  libavfilter/vaf_spectrumsynth.c |2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavfilter/vaf_spectrumsynth.c
> > b/libavfilter/vaf_spectrumsynth.c
> > index ab9a69b..ba14d8d 100644
> > --- a/libavfilter/vaf_spectrumsynth.c
> > +++ b/libavfilter/vaf_spectrumsynth.c
> > @@ -442,11 +442,11 @@ static int try_push_frames(AVFilterContext *ctx)
> >  case SCROLL:
> >  s->xpos = s->xend - 1;
> >  ret = try_push_frame(ctx, s->xpos);
> > +break;
> >  case RSCROLL:
> >  s->xpos = 0;
> >  ret = try_push_frame(ctx, s->xpos);
> >  break;
> > -break;
> >  case FULLFRAME:
> >  for (x = 0; x < s->xend; x++) {
> >  ret = try_push_frame(ctx, x);
> > --
> > 1.7.9.5
> >
> > ___
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> 
> LGTM, silly mistake.

applied

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter/vaf_spectrumsynth: Move "break" up

[Paul B Mahol]

On 2/7/16, Michael Niedermayer  wrote:
> Fixes CID1351347
>
> Signed-off-by: Michael Niedermayer 
> ---
>  libavfilter/vaf_spectrumsynth.c |2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavfilter/vaf_spectrumsynth.c
> b/libavfilter/vaf_spectrumsynth.c
> index ab9a69b..ba14d8d 100644
> --- a/libavfilter/vaf_spectrumsynth.c
> +++ b/libavfilter/vaf_spectrumsynth.c
> @@ -442,11 +442,11 @@ static int try_push_frames(AVFilterContext *ctx)
>  case SCROLL:
>  s->xpos = s->xend - 1;
>  ret = try_push_frame(ctx, s->xpos);
> +break;
>  case RSCROLL:
>  s->xpos = 0;
>  ret = try_push_frame(ctx, s->xpos);
>  break;
> -break;
>  case FULLFRAME:
>  for (x = 0; x < s->xend; x++) {
>  ret = try_push_frame(ctx, x);
> --
> 1.7.9.5
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

LGTM, silly mistake.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel