[FFmpeg-devel] [PATCH] mss1: improve check if decoded pivot is invalid

Paul B Mahol onemda at gmail.com
Sun Jul 15 04:16:19 CEST 2012

On 7/15/12, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Sun, Jul 15, 2012 at 12:50:50AM +0000, Paul B Mahol wrote:
>> On 7/15/12, Derek Buitenhuis <derek.buitenhuis at gmail.com> wrote:
>> > On 14/07/2012 8:39 PM, Paul B Mahol wrote:
>> >>> Can you add an actual explanation of what you fixed
>> >>> > into the commit message?
>> >> You mean bug report or explanation what that C code does?
>> >
>> > In what way does it 'improve' it? What does it fix? You
>> > don't even have a bug tracker ID. This commit is completely
>> > missing any useful information at all.
>> Get over it. I had enough of such hypocrite reviews with Libav.
> I think derek just wants a more elaborate commit message that explains
> with more detail what this fixes and why and how.
> Maybe something like:
> the pivot has to lie between 0 and base thus a check of ==base is
> insufficient. Thus replace it by a proper check
> IMHO adding that extra line is not a bad idea, though its no major
> thing to me if that line is there or not. Its a major thing to others
> it seems ...

Pushed with that explanation.

More information about the ffmpeg-devel mailing list