[FFmpeg-devel] [PATCH] avcodec/dv_profile: deprecate internal function that shouldn't be public

Michael Niedermayer michaelni at gmx.at
Fri Sep 19 11:58:53 CEST 2014


On Fri, Sep 19, 2014 at 01:50:28AM -0300, James Almer wrote:
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> Unlike us, libav got rid of the internal symbols before the header became public on 
> an actual relase, so we're stuck with it until next bump.
> I renamed the function to ff_ since it's only used outside libavcodec when called
> with the public symbol.
> 
>  doc/APIchanges                   |  3 +++
>  libavcodec/dv_profile.c          | 12 ++++++++++--
>  libavcodec/dv_profile.h          |  6 ++++++
>  libavcodec/dv_profile_internal.h |  8 ++++++++
>  libavcodec/dvdec.c               |  3 ++-
>  libavcodec/version.h             |  5 ++++-
>  6 files changed, 33 insertions(+), 4 deletions(-)

LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20140919/c6dc5865/attachment.asc>


More information about the ffmpeg-devel mailing list