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
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
