[MPlayer-dev-eng] [PATCH] minor cosmetic stream_dvd.c simplification

Nico Sabbi nsabbi at email.it
Thu Feb 15 15:53:06 CET 2007


Reimar Döffinger wrote:

>Hello,
>On Thu, Feb 15, 2007 at 01:46:55PM +0100, Reimar D?ffinger wrote:
>  
>
>>I know the compiler can do common subexpression elimination, but I
>>always find it good when the source code is easier to read than the
>>compiled one.
>>Having at least some lines shorter than 80 characters really helps I
>>think...
>>I intend to apply soon if noone objects.
>>I also found some stupidity: the audio channel "checking" code tests for
>>if(vts_file->vts_pgcit) != 0, but this is pointless since the equivalent
>>subtitle code does not. Probably it can simply be removed...
>>    
>>
>
>Sorry, forgot to re-diff to get the actually working version...
>
>Greetings,
>Reimar Döffinger
>  
>

if it works it's fine

 
 
 --
 Email.it, the professional e-mail, gratis per te: http://www.email.it/f
 
 Sponsor:
 Le speciali Offerte di Benvenuto di Cassine di Pietra:
* scopra il gusto ed i vantaggi delle tradizioni contadine
* 
 Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid924&d-2



More information about the MPlayer-dev-eng mailing list