<div name="BB10" id="BB10_response_div" dir="auto" contenteditable="true"
style="width:100%;background:"#ffffff"; font-size:
initial;font-family:"Calibri","Slate
Pro","sans-serif";color:#1f497d;"><br
style="display:initial"></div>
<div name="BB10" id="response_div_spacer" dir="auto" contenteditable="true"
style="width:100%;background:"#ffffff"; font-size:
initial;font-family:"Calibri","Slate
Pro","sans-serif";color:#1f497d;"><br
style="display:initial"></div>
<div id="_signaturePlaceholder" name="BB10" dir="auto"
contenteditable="true"
style="font-size:
initial;font-family:"Calibri","Slate
Pro","sans-serif";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:"Slate Pro"">
Original Message </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:"Tahoma","BB Alpha Sans","Slate
Pro","sans-serif";"> <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 <vigne...@google.com> wrote:<br/><br/>>
avio_seek(matroska->ctx->pb, before_pos, SEEK_SET);<br/>> + if
(cues_start == -1 || cues_end == -1) return -1;<br/><br/>Shouldn'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