On Mon, Mar 3, 2014 at 10:47 PM, Anton Khirnov <[email protected]> wrote:
> get_channel() returns 0 on success
>
> CC:[email protected]
> ---
> libavfilter/af_channelmap.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavfilter/af_channelmap.c b/libavfilter/af_channelmap.c
> index 71d51e7..6bc8c3a 100644
> --- a/libavfilter/af_channelmap.c
> +++ b/libavfilter/af_channelmap.c
> @@ -188,7 +188,7 @@ static av_cold int channelmap_init(AVFilterContext *ctx)
> s->map[i].out_channel_idx = i;
> break;
> case MAP_ONE_STR:
> - if (!get_channel(&mapping, &in_ch, separator)) {
> + if (!get_channel(&mapping, &in_ch, separator) < 0) {
Shouldn't this be
if (get_channel(&mapping, &in_ch, separator) < 0) {
(without !)?
> av_log(ctx, AV_LOG_ERROR, err);
> return AVERROR(EINVAL);
> }
> --
> 1.7.10.4
>
> _______________________________________________
> libav-devel mailing list
> [email protected]
> https://lists.libav.org/mailman/listinfo/libav-devel
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
