[FFmpeg-devel] [PATCH]lavc/avpacket: Make pkt parameter of av_packet_get_side_data() const
Carl Eugen Hoyos
ceffmpeg at gmail.com
Tue Mar 7 10:59:42 EET 2017
2017-02-26 12:04 GMT+01:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
> 2017-02-26 11:51 GMT+01:00 Nicolas George <george at nsup.org>:
>>> -uint8_t* av_packet_get_side_data(AVPacket *pkt, enum AVPacketSideDataType type,
>>> +uint8_t* av_packet_get_side_data(
>>> +#if FF_API_CONST_GET_SIDE_DATA
>>> + AVPacket *pkt, enum AVPacketSideDataType type,
>> I do not think we need the FF_API dance, since it is not an ABI break.
> I agree that there is no ABI break (but I suspect FF_API does not
> imply an ABI break).
>> I think is is not an API break either.
> I may misremember but I thought it's an API break for c++ users.
> Anyway, new patch attached.
Should this not be changed, should one of the patches be applied?
Thank you, Carl Eugen
More information about the ffmpeg-devel