diff options
| author | Diego Pettenò <flameeyes@gmail.com> | 2008-10-02 19:24:26 +0000 |
|---|---|---|
| committer | Guillaume Poirier <gpoirier@mplayerhq.hu> | 2008-10-02 19:24:26 +0000 |
| commit | 9aa1cfec1a8597f3bd0f9fa564c43a53e42fab25 (patch) | |
| tree | e30139853604264d2b354d1db3c364d8ed701526 | |
| parent | 2d5174fc46de0cc6452290b901e0a68b729a6704 (diff) | |
| download | ffmpeg-9aa1cfec1a8597f3bd0f9fa564c43a53e42fab25.tar.gz | |
fix warning reported by Intel C compiler:
libavcodec/h264_parser.c(77): warning #589: transfer of control
bypasses initialization of:
variable "v" (declared at line 58)
goto found;
This new form also improves readability.
Patch by Diego 'Flameeyes' Pettenò %flameeyes A gmail P com%
Originally committed as revision 15526 to svn://svn.ffmpeg.org/ffmpeg/trunk
| -rw-r--r-- | libavcodec/h264_parser.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index 7a85d770cf..73b156964c 100644 --- a/libavcodec/h264_parser.c +++ b/libavcodec/h264_parser.c @@ -59,10 +59,7 @@ int ff_h264_find_frame_end(H264Context *h, const uint8_t *buf, int buf_size) if(v==7 || v==8 || v==9){ if(pc->frame_start_found){ i++; -found: - pc->state=7; - pc->frame_start_found= 0; - return i-(state&5); + goto found; } }else if(v==1 || v==2 || v==5){ if(pc->frame_start_found){ @@ -80,6 +77,11 @@ found: } pc->state= state; return END_NOT_FOUND; + +found: + pc->state=7; + pc->frame_start_found= 0; + return i-(state&5); } static int h264_parse(AVCodecParserContext *s, |