[MPlayer-dev-eng] [PATCH] Re: min() and max() macro definitions

Dominik 'Rathann' Mierzejewski dominik at rangers.eu.org
Sat Mar 19 17:28:16 CET 2005


On Saturday, 19 March 2005 at 09:36, Nico Sabbi wrote:
[...]
> as far as I'm concerned the patches to demux_ts.c, demux_mpg.c, 
> muxer_mpg.c, mpeg_hdr.c and dvbin.c are ok to commit (I'll leave the
> rest to the other mainteners), but do we really need all this?

I think it's bad coding to have max() and min() defined multiple times all
over the place.

R.

-- 
MPlayer RPMs maintainer: http://rpm.greysector.net/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"




More information about the MPlayer-dev-eng mailing list