diff mbox

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

Message ID 20190429144752.826-1-onemda@gmail.com
State Accepted
Commit 31990046ac68f218efed846931824adab9d51370
Headers show

Commit Message

Paul B Mahol April 29, 2019, 2:47 p.m. UTC 
Signed-off-by: Paul B Mahol <onemda@gmail.com>
---
 libavfilter/f_ebur128.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Clément Bœsch April 29, 2019, 3:01 p.m. UTC | #1 
On Mon, Apr 29, 2019 at 04:47:52PM +0200, Paul B Mahol wrote:
> Signed-off-by: Paul B Mahol <onemda@gmail.com>
> ---
>  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
diff mbox

Patch

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) {