[FFmpeg-cvslog] r16776 - trunk/libavutil/mem.c

Diego Biurrun diego
Sun Jan 25 20:31:12 CET 2009


On Sun, Jan 25, 2009 at 08:17:59PM +0100, Michael Niedermayer wrote:
> On Sun, Jan 25, 2009 at 07:10:06PM +0100, Reimar D?ffinger wrote:
> > On Sun, Jan 25, 2009 at 07:05:57PM +0100, Reimar D?ffinger wrote:
> > > On Sun, Jan 25, 2009 at 06:36:48PM +0100, diego wrote:
> > > > 
> > > > Log:
> > > > Do not #include common.h without necessity.
> > > > This eliminates the need to work around common.h overriding certain system
> > > > functions which are required in this header.
> > > 
> > > And how is that supposed to work when someone includes common.h before
> > > mem.h? Both are part of the public API.
> > 
> > How silly of me, I did not see this is mem.c, not mem.h.
> > It does lead to another question, why is mem.h not included in mem.c?
> 
> because diego removed it, or more precissely it was included over common.h

So what do you want done?

> anyway, i also must say that iam rather unhappy about the whole header
> changes being commited without prior patches

Noted.

Diego




More information about the ffmpeg-cvslog mailing list