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

[Michael Niedermayer]

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.


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090125/f5fcfa67/attachment.pgp>