[FFmpeg-devel] [PATCH] avcodec/imgconvert: remove deprecation guards for a function that's not declared as such
jamrial at gmail.com
Fri Aug 25 18:12:15 EEST 2017
On 8/25/2017 11:39 AM, Ronald S. Bultje wrote:
> On Fri, Aug 25, 2017 at 10:04 AM, James Almer <jamrial at gmail.com> wrote:
>> On 8/25/2017 8:10 AM, Ronald S. Bultje wrote:
>>> On Thu, Aug 24, 2017 at 7:43 PM, James Almer <jamrial at gmail.com> wrote:
>>>> Signed-off-by: James Almer <jamrial at gmail.com>
>>>> The deprecation seems to have been imported from libav but never made
>>> Hm, but do we really want this function? Shouldn't all users be ported to
>>> the function this wraps?
>> I don't know. The Doxy even acknowledges there's an alternative but
>> mentions its use case is apparently different, which is probably why
>> the deprecation wasn't made effective.
> We should all acknowledge fully that there is a use case for
> memcpy_inverted(source, destination, size), yet libc does not define it. I
> don't know why. It's silly. I feel libc is being racist.
> Silliness aside, let's not have multiple APIs that do the same thing.
> If the function really needs to go, then the deprecated attribute should
>> be added to the header. And from that point the ~2 years deprecation
>> period starts.
> Sure, operationally I don't really care, as long as the end product is that
> it goes away.
Ok, patch dropped and a new one sent that effectively deprecates the
> Fork nastiness aside, I feel that one thing Libav does have a much better
> handle on than us is API cleanliness.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel