[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