[FFmpeg-devel] [PATCH 1/8] avcodec/old_codec_ids: workaround doxygen inclusion bug

Michael Niedermayer michaelni at gmx.at
Sun Jul 6 00:42:28 CEST 2014


On Sat, Jul 05, 2014 at 02:38:48PM -0700, Timothy Gu wrote:
> On Jul 4, 2014 11:06 AM, "Timothy Gu" <timothygu99 at gmail.com> wrote:
> >
> > Currently, http://ffmpeg.org/doxygen/trunk/group__preproc__misc.html is
> > broken, because of inclusion bug in Doxygen's preprocessing mode. The
> > now-unincluded header file is included anyway in avcodec.h, the only
> > place where it is used, so there should be no problem removing it.
> >
> > One concern is API-compatibility, because old_codec_ids.h is a public
> > header. However, IMO this is not a problem because currently users
> > cannot include only this header and not `avcodec.h` anyway, because of
> > missing AV_CODEC_ID_NONE symbol.
> >
> > Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
> > ---
> >  libavcodec/old_codec_ids.h | 2 --
> >  1 file changed, 2 deletions(-)
> 
> Ping.

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140706/e6eae5b7/attachment.asc>


More information about the ffmpeg-devel mailing list