On Mon, Aug 06, 2018 at 19:51:48 +0530, Gyan Doshi wrote: > > > On 06-08-2018 07:13 PM, hwren wrote: > > > - .long_name = NULL_IF_CONFIG_SMALL("Decoder for AVS2/IEEE 1857.4"), > > + .long_name = NULL_IF_CONFIG_SMALL("Decoder for > > AVS2-P2/IEEE1857.4"), > > "Decoder for" is not needed. The flag indicates its role in `ffmpeg > -codecs`. > > Rest docs LGTM
And the change of codec description doesn't fit with the patch's commit message. It should probably be split out into two patches, or the commit message amended. Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel