[MPlayer-dev-eng] [PATCH] matroska chapter seeking

Evgeniy Stepanov eugeni.stepanov at gmail.com
Sat Jul 15 20:54:59 CEST 2006


On Saturday 15 July 2006 21:21, Nico Sabbi wrote:
> >+  demuxer->chapters = mkv_d->chapters;
> >+  demuxer->num_chapters = mkv_d->num_chapters;
> >+
>
> I really don't like those two lines; instead you should memcpy (or
> assign directly to demuxer)
> the chapters structure

Agreed, no point in keeping chapters in mkv_d. Moved to demuxer.

> You totally missed the uninit part of demuxer->chapters :)

It was not needed since only the pointer was copied to demuxer.

Man page updated.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: seek_chapter.patch
Type: text/x-diff
Size: 12630 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060715/0815e783/attachment.patch>


More information about the MPlayer-dev-eng mailing list