On 12 Mar 2014, at 16:27, Vittorio Giovara <vittorio.giov...@gmail.com> wrote:
> On Wed, Mar 12, 2014 at 4:23 PM, Tim Walker <tdskywal...@gmail.com> wrote:
>> On 12 Mar 2014, at 15:30, Vittorio Giovara <vittorio.giov...@gmail.com>
>> wrote:
>>
>>> ---
>>> libavcodec/ljpegenc.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/libavcodec/ljpegenc.c b/libavcodec/ljpegenc.c
>>> index 7eb4675..7c1b3bc 100644
>>> --- a/libavcodec/ljpegenc.c
>>> +++ b/libavcodec/ljpegenc.c
>>> @@ -329,6 +329,6 @@ AVCodec ff_ljpeg_encoder = {
>>> AV_PIX_FMT_BGR24,
>>> AV_PIX_FMT_YUV420P,
>>> AV_PIX_FMT_YUV422P,
>>> - AV_PIX_FMT_YUVJ444P,
>>> + AV_PIX_FMT_YUV444P,
>>> AV_PIX_FMT_NONE },
>>> };
>>> --
>>> 1.8.3.4 (Apple Git-47)
>>
>> I fail to see the correlation between the patch and the commit message.
>>
>
> Amended to ‘fix duplicated pixel format entry’.
> Vittorio
I would prefer something like: "ljpeg: fix typo in pixel format list", since
this is what the patch does. Else make two patches, "ljpeg: remove duplicate
pixel format entry" and "ljpeg: add missing pixel format entry".
Tim
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
