[MPlayer-dev-eng] [PATCH] MinGW compile fix, header order policy suggestion

Ivan Kalvachev ikalvachev at gmail.com
Wed Jun 1 00:30:56 CEST 2005


On 5/31/05, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> Hi,
> On Tue, May 31, 2005 at 12:26:55PM +0300, Ivan Kalvachev wrote:
> > I don't think this needs to be enforced because it is an good
> > programer writing style.
> 
> Well, that probably means we have very few good programmers? The order

Not all good programners have good programing style.
And not all that have good programing style are good programmers.

> is often different - which isn't so bad, except for the fact that even
> in cases where there is a good reason for it it is not explained...

> > As for the current situation, fixing this way is even worse than not
> > fixing it. The right way is to avoid all possible collisions and
> > change STREAM_SEEK to MP_STREAM_SEEK. Otherwise the assigned value may
> 
> But for consistency all STREAM_* defines would have to be renamed...
> That would be a gigantic patch!

Yes.
FFmpeg already passed such edition.
 
> > be same as another defined STREAM_* label that would be produce an
> > VERY hard to track bugs. The only clue we may find for this bug, is an
> > warning that is hidden in the 200k warnings mplayer emits. There could
> > be an even worse situation as STREAM_SEEK may not be defined as value
> > or constant...
> 
> As I said, in this case STREAM_SEEK is a typedef'ed struct in the
> windows headers, so this can only lead to compile errors, so it's not
> quite as bad...

Yes, i read it the first time, but I didn't believed on my poor
english skills, because if that was true  then this fix won't fix
anything....




More information about the MPlayer-dev-eng mailing list