[FFmpeg-devel] [PATCH] avfilter/af_channelsplit: add channels option

[Nicolas George]

Paul B Mahol (2018-03-21):
> >> +        uint64_t channel_layout;
> >> +        int nb_extracted_channels;
> > Inconsistent variable names.
> I dont follow.

The two variables describe the same entity, they should have the same
base name: nb_extracted_channels and extracted_channels, obviously.

> Where do you see I copy pasted anything?

The block I quoted is almost identical to the block just before.
Duplicated code is unacceptable. Factor it.

> Accepted stuff fixed locally and gonna apply.

Send the updated patch for review. But fix the duplicated code before.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180321/7f15c09d/attachment.sig>