[MPlayer-cvslog] r25894 - trunk/libmpcodecs/native/alaw.h

Uoti Urpala uoti.urpala at pp1.inet.fi
Sun Jan 27 20:46:32 CET 2008


On Sun, 2008-01-27 at 20:32 +0100, Roberto Togni wrote:
> On Sun, 27 Jan 2008 19:49:25 +0100 (CET)
> diego <subversion at mplayerhq.hu> wrote:
> > Replace with the output of the updated alaw-gen generator program.
> > This adds multiple inclusion guards and reformats the tables.

> WTF?
> 
> There is no need to reformat the file to add multiple inclusion
> protection.

I think the commit message was meant to say that the file was
reformatted as a side effect of using the output of the current
generator program.

> And the table was already aligned.
> 
> Please revert.

Why revert? Is the new table somehow worse? Even if you think the change
was pointless, changing it back and forth would be doubly so.




More information about the MPlayer-cvslog mailing list