[FFmpeg-devel] GSOC update on the CDXL demuxer

[Erion Omeri]

Yesterday I got the Sound part working.
I have removed the preprocessor marks for enabling debuggin and replaced
them with a simple IF statement, and will remove them all before submitting
my files.
I am now working on the HAM6 decoder for the CDXL Video.

Erion

On Wed, Apr 15, 2009 at 7:57 AM, Reimar D?ffinger
<Reimar.Doeffinger at gmx.de>wrote:

> On Wed, Apr 15, 2009 at 02:01:49PM +0200, Diego Biurrun wrote:
> > On Tue, Apr 14, 2009 at 08:07:54PM +0200, Reimar D?ffinger wrote:
> > > On Tue, Apr 14, 2009 at 11:39:59AM -0500, Erion Omeri wrote:
> > >
> > > >     #if 0
> > > >     debug_header(cdxl);
> > > >     #endif
> > >
> > > The # of preprocessor directives must be the first character in the
> > > line. If you want to indent them you'd do it like this:
> > > > #   if 0
> >
> > This is not true, the # does not have to be on the first column.
>
> Yes, I should have said "should be the first character in the line,
> if it's not it might fail with some compilers, produce warnings with
> others, some syntax-highlighting editors don't handle it right and
> generally makes those directives hard to spot".
> Just thought a must would be a reasonable way to summarize the idea,
> despite the fact that the C99 standard in its examples indeed places
> the spaces before the #.
> But while we're at it I'll also say that indenting preprocessing
> directives to the code level does not really make sense since they
> are processed independently, indentation makes only sense for
> preprocessing directives within other preprocessing directives.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>