[FFmpeg-devel] [PATCH] lavf/matroskadec: Ensure cues_end is initialized
ooe at st-andrews.ac.uk
ooe at st-andrews.ac.uk
Tue Aug 26 15:33:56 CEST 2014
<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 <vigneshv at 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 at ffmpeg.org<br/>http://ffmpeg.org/mailman/listinfo/ffmpeg-devel<br/>
More information about the ffmpeg-devel
mailing list