Re: [FFmpeg-devel] [PATCH] avfilter/f_ebur128: use correct type for chl

[Clément Bœsch]

On Mon, Apr 29, 2019 at 04:47:52PM +0200, Paul B Mahol wrote:
> Signed-off-by: Paul B Mahol 
> ---
>  libavfilter/f_ebur128.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavfilter/f_ebur128.c b/libavfilter/f_ebur128.c
> index f613d8def2..f25d5f096e 100644
> --- a/libavfilter/f_ebur128.c
> +++ b/libavfilter/f_ebur128.c
> @@ -420,7 +420,7 @@ static int config_audio_output(AVFilterLink *outlink)
>  
>  for (i = 0; i < nb_channels; i++) {
>  /* channel weighting */
> -const uint16_t chl = 
> av_channel_layout_extract_channel(outlink->channel_layout, i);
> +const uint64_t chl = 
> av_channel_layout_extract_channel(outlink->channel_layout, i);
>  if (chl & (AV_CH_LOW_FREQUENCY|AV_CH_LOW_FREQUENCY_2)) {
>  ebur128->ch_weighting[i] = 0;
>  } else if (chl & BACK_MASK) {

LGTM, thanks

-- 
Clément B.


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH] avfilter/f_ebur128: use correct type for chl

[Paul B Mahol]

Signed-off-by: Paul B Mahol 
---
 libavfilter/f_ebur128.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavfilter/f_ebur128.c b/libavfilter/f_ebur128.c
index f613d8def2..f25d5f096e 100644
--- a/libavfilter/f_ebur128.c
+++ b/libavfilter/f_ebur128.c
@@ -420,7 +420,7 @@ static int config_audio_output(AVFilterLink *outlink)
 
 for (i = 0; i < nb_channels; i++) {
 /* channel weighting */
-const uint16_t chl = 
av_channel_layout_extract_channel(outlink->channel_layout, i);
+const uint64_t chl = 
av_channel_layout_extract_channel(outlink->channel_layout, i);
 if (chl & (AV_CH_LOW_FREQUENCY|AV_CH_LOW_FREQUENCY_2)) {
 ebur128->ch_weighting[i] = 0;
 } else if (chl & BACK_MASK) {
-- 
2.17.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".