On Wed, Mar 12, 2014 at 4:33 PM, Tim Walker <tdskywal...@gmail.com> wrote:
> 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".
>
The first log is fine.
CC: libav-stable
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
