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

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Fri Feb 16 16:03:40 CET 2007


Hello,
On Thu, Feb 15, 2007 at 01:51:01PM +0100, Reimar D?ffinger wrote:
> 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...

Applied btw.

Greetings,
eimar Döffinger



More information about the MPlayer-dev-eng mailing list