[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