[FFmpeg-devel] [PATCH] avs2: Correct expression error and simplify log
Moritz Barsnick
barsnick at gmx.net
Tue Aug 7 13:08:23 EEST 2018
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
More information about the ffmpeg-devel
mailing list