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

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Jan 27 20:41:55 CET 2008


Hello,
On Sun, Jan 27, 2008 at 08:32:56PM +0100, Roberto Togni wrote:
> On Sun, 27 Jan 2008 19:49:25 +0100 (CET)
> diego <subversion at mplayerhq.hu> wrote:
> 
> > Author: diego
> > Date: Sun Jan 27 19:49:24 2008
> > New Revision: 25894
> > 
> > Log:
> > 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.

Well, I guess this is importing FFmpeg policy that all headers should
have one unless they are supposed to be included multiple time, for
consistency and to avoid problems later.

> And the table was already aligned.

Without spaces between , and - though.

> Please revert.

Seems rather pointless to me, does it hurt in any way?



More information about the MPlayer-cvslog mailing list