[MPlayer-dev-eng] [PATCH] dvdnav part 3

Nico Sabbi nsabbi at email.it
Wed Nov 22 15:04:31 CET 2006


Ötvös Attila wrote:

>szerda 22 november 2006 10:24 dátummal Diego Biurrun ezt írta:
>  
>
>>I just had a very quick look, so I can only comment on formalities that
>>caught my eye:
>>
>>- Please don't compress tiny patches, it just makes them bothersome to
>>  handle and reviews less likely.
>>- Nit: Please don't add trailing whitespace.
>>- Please don't mix spaces and tabs for indentation.
>>- Please respect surrounding indentation, i.e. don't indent by two
>>  spaces when most of the code in mplayer.c is indented by four.
>>
>>Other than that it would help if you could split your patches into small
>>self-contained parts.
>>    
>>
>
>I corrected and separated.
>
>Comment?
>
>Best regards.
>Attila
>  
>
>  
>
just a very quick look:

>+#define VDCTRL_SET_CODEC_FLAG_LOW_DELAY 10 /* set/reset codec flag low delay */
>  
>

this

> 
>+
>+void vd_set_codec_flag_low_delay(sh_video_t *sh_video, int mode) {
>+if (mpvdec)
>+    mpvdec->control(sh_video,VDCTRL_SET_CODEC_FLAG_LOW_DELAY, (void*)(&mode));
>+return;
>+}
>  
>

and this should not be necessary. I found the proper patch for lavc; 
I'll test it tonight
if it works as expected.
Can someone give a look at Attila's SPU code, please?





More information about the MPlayer-dev-eng mailing list