[MPlayer-cvslog] r26111 - trunk/mp_msg.h

Diego Biurrun diego at biurrun.de
Sat Mar 1 11:38:23 CET 2008


On Thu, Feb 28, 2008 at 10:53:06PM -0500, Rich Felker wrote:
> On Thu, Feb 28, 2008 at 07:31:35PM +0100, diego wrote:
> > 
> > Log:
> > TARGET_OS2 is never set, use __OS2__ instead.
> > 
> > --- trunk/mp_msg.h	(original)
> > +++ trunk/mp_msg.h	Thu Feb 28 19:31:35 2008
> > @@ -111,7 +111,7 @@ int mp_msg_test(int mod, int lev);
> >  
> > -#ifdef TARGET_OS2
> > +#ifdef __OS2__
> 
> Again, I'm against this general class of change. I've explained the
> reasoning many times! Would you please honor that or at least discuss
> it rather than continuing these bad commits?

I've discussed it before, I get tired of repeating it: I have little
interest in discussing your theories.  There is no practical alternative
to what I committed.

If you disagree: Patches welcome.

Diego



More information about the MPlayer-cvslog mailing list