Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Ensure cues_end is initialized

[ooe]

<div name="BB10" id="BB10_response_div" dir="auto" contenteditable="true"       
                                                                                
      style="width:100%;background:&quot;#ffffff&quot;; font-size: 
initial;font-family:&quot;Calibri&quot;,&quot;Slate 
Pro&quot;,&quot;sans-serif&quot;;color:#1f497d;"><br 
style="display:initial"></div>                                                  
                                                                                
   <div name="BB10" id="response_div_spacer" dir="auto" contenteditable="true"  
                                                                                
         style="width:100%;background:&quot;#ffffff&quot;; font-size: 
initial;font-family:&quot;Calibri&quot;,&quot;Slate 
Pro&quot;,&quot;sans-serif&quot;;color:#1f497d;"><br 
style="display:initial"></div>                                                  
                                                                                
   <div id="_signaturePlaceholder" name="BB10" dir="auto"  
contenteditable="true"                                                          
                            style="font-size: 
initial;font-family:&quot;Calibri&quot;,&quot;Slate 
Pro&quot;,&quot;sans-serif&quot;;color:#1f497d;"></div>                         
              <table id="_bb10TempSeparator" width="100%" 
style="background-color:white; border-spacing:0px;" contenteditable="false">    
                                           <tr><td></td><td 
id="_separatorInternal" rowspan=2 style="text-align:center">                    
                                                                        <span 
id="_bb10TempSeparatorText" style="background-color:white; 
color:#0073BC;font-size:smaller;font-family:&quot;Slate Pro&quot;">&nbsp; 
Original Message &nbsp;</span>      </td></tr>                                  
                                                                                
                                             <tr> <td colspan=2><div 
style="border:none;border-top:solid #0073BC 1.0pt;">                            
                                                               
</td></tr></table>                                                              
                                                                                
   <table width="100%" style="background-color:white;border-spacing:0px;"> 
<tr><td id="_persistentHeaderContainer" colspan=2>                              
                <div id="_persistentHeader" style="font-size: 
smaller;font-family:&quot;Tahoma&quot;,&quot;BB Alpha Sans&quot;,&quot;Slate 
Pro&quot;,&quot;sans-serif&quot;;">  <div id=From ><b>From: </b>Reimar 
Döffinger</div><div id=Sent ><b>Sent: </b>Tuesday, 26 August 2014 
08:31</div><div id=To ><b>To: </b>FFmpeg development discussions and 
patches</div><div id=ReplyTo ><b>Reply To: </b>FFmpeg development discussions 
and patches</div><div id=Subject ><b>Subject: </b>Re: [FFmpeg-devel] [PATCH] 
lavf/matroskadec: Ensure cues_end is 
initialized</div></div></td></tr></table><div id="_persistentHeaderEnd" 
style="border:none;border-top:solid #babcd1 1pt;"></div><br><div name="BB10" 
dir="auto" id="_originalContent"><br/><br/>On 25.08.2014, at 18:15, Vignesh 
Venkatasubramanian &lt;vigne...@google.com&gt; wrote:<br/><br/>&gt;     
avio_seek(matroska-&gt;ctx-&gt;pb, before_pos, SEEK_SET);<br/>&gt; +    if 
(cues_start == -1 || cues_end == -1) return -1;<br/><br/>Shouldn&apos;t this 
use something more descriptive than 
-1?<br/>_______________________________________________<br/>ffmpeg-devel 
mailing 
list<br/>ffmpeg-devel@ffmpeg.org<br/>http://ffmpeg.org/mailman/listinfo/ffmpeg-devel<br/>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel