On 25.08.2014, at 18:15, Vignesh Venkatasubramanian <vigneshv at google.com> wrote: > avio_seek(matroska->ctx->pb, before_pos, SEEK_SET); > + if (cues_start == -1 || cues_end == -1) return -1; Shouldn't this use something more descriptive than -1?