On 6/27/19, Liu Steven <[email protected]> wrote:
>
>
>> 在 2019年6月27日,下午4:37,Paul B Mahol <[email protected]> 写道:
>>
>> On 6/27/19, Paul B Mahol <[email protected]> wrote:
>>> On 6/27/19, Steven Liu <[email protected]> wrote:
>>>> Signed-off-by: Steven Liu <[email protected]>
>>>> ---
>>>> libavcodec/cfhd.c | 14 --------------
>>>> 1 file changed, 14 deletions(-)
>>>>
>>>> diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c
>>>> index 846d334b9b..616f5af193 100644
>>>> --- a/libavcodec/cfhd.c
>>>> +++ b/libavcodec/cfhd.c
>>>> @@ -136,20 +136,6 @@ static inline void peak_table(int16_t *band, Peak
>>>> *peak, int length)
>>>>             band[i] = bytestream2_get_le16(&peak->base);
>>>> }
>>>>
>>>> -static inline void process_alpha(int16_t *alpha, int width)
>>>> -{
>>>> -    int i, channel;
>>>> -    for (i = 0; i < width; i++) {
>>>> -        channel   = alpha[i];
>>>> -        channel  -= ALPHA_COMPAND_DC_OFFSET;
>>>> -        channel <<= 3;
>>>> -        channel  *= ALPHA_COMPAND_GAIN;
>>>> -        channel >>= 16;
>>>> -        channel   = av_clip_uintp2(channel, 12);
>>>> -        alpha[i]  = channel;
>>>> -    }
>>>> -}
>>>> -
>>>> static inline void filter(int16_t *output, ptrdiff_t out_stride,
>>>>                           int16_t *low, ptrdiff_t low_stride,
>>>>                           int16_t *high, ptrdiff_t high_stride,
>>>> --
>>>> 2.17.2 (Apple Git-113)
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> ffmpeg-devel mailing list
>>>> [email protected]
>>>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>>
>>>> To unsubscribe, visit link above, or email
>>>> [email protected] with subject "unsubscribe".
>>>
>>> NAK
>>>
>>
>> This code is needed for processing alpha, and should not be removed.
> I remove it because i cannot found the caller for this API.
> I think there should have one caller for this API, It will not be used if
> there have no caller, Isn’t it?

This is not an API. Please leave this function alone. If you really
need to fix it,
look at prior commits to this file and re-add alpha support.

>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> [email protected]
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> [email protected] with subject "unsubscribe".
>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> [email protected]
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> [email protected] with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to