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

[Diego Biurrun]

On Sun, Jan 25, 2009 at 08:38:14PM +0100, Michael Niedermayer wrote:
> On Sun, Jan 25, 2009 at 08:31:12PM +0100, Diego Biurrun wrote:
> > 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?
> 
> for all c files for which there exist a equally named header file that header
> should explicitly be #included in the C file.

Done for libavutil.

Diego