[FFmpeg-cvslog] r15526 - trunk/libavcodec/h264_parser.c

[gpoirier]

Author: gpoirier
Date: Thu Oct  2 21:24:26 2008
New Revision: 15526

Log:
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%


Modified:
   trunk/libavcodec/h264_parser.c

Modified: trunk/libavcodec/h264_parser.c
==============================================================================
--- trunk/libavcodec/h264_parser.c	(original)
+++ trunk/libavcodec/h264_parser.c	Thu Oct  2 21:24:26 2008
@@ -59,10 +59,7 @@ int ff_h264_find_frame_end(H264Context *
             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,