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

Reimar Döffinger Reimar.Doeffinger
Sun Jan 25 19:05:57 CET 2009


On Sun, Jan 25, 2009 at 06:36:48PM +0100, diego wrote:
> Author: diego
> Date: Sun Jan 25 18:36:48 2009
> New Revision: 16776
> 
> 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.
I really don't consider it good style to break a header that is part of
the public API even further when the original breakage was not even
fixed, and I request that at least for now _absolutely_ _no_ changes to
public headers are made without sending a patch first.

Greetings,
Reimar D?ffinger




More information about the ffmpeg-cvslog mailing list