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

Nico Sabbi nicola_sabbi at fastwebnet.it
Sat Jul 15 21:16:36 CEST 2006


Evgeniy Stepanov wrote:

>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.
>  
>
>------------------------------------------------------------------------
>

looks nice to me



More information about the MPlayer-dev-eng mailing list