fix warning reported by Intel C compiler:
authorDiego Pettenò <flameeyes@gmail.com>
Thu, 2 Oct 2008 19:24:26 +0000 (19:24 +0000)
committerGuillaume Poirier <gpoirier@mplayerhq.hu>
Thu, 2 Oct 2008 19:24:26 +0000 (19:24 +0000)
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

libavcodec/h264_parser.c

index 7a85d77..73b1569 100644 (file)
@@ -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,