[FFmpeg-devel] [PATCH] Kega Game Video (KGV1) decoder

Daniel Verkamp daniel
Tue Mar 9 02:03:26 CET 2010


On Mon, Mar 8, 2010 at 5:45 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Wed, Mar 03, 2010 at 07:48:43PM -0600, Daniel Verkamp wrote:
>> On Wed, Mar 3, 2010 at 5:47 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> > On Wed, Mar 03, 2010 at 05:23:32PM -0600, Daniel Verkamp wrote:
>> >> On Tue, Mar 2, 2010 at 4:37 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> >> > On Tue, Mar 02, 2010 at 11:46:38AM -0600, Daniel Verkamp wrote:
>> >> >> On Wed, Feb 3, 2010 at 9:02 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> >> >> > On Wed, Feb 03, 2010 at 11:49:32AM -0600, Daniel Verkamp wrote:
>> >> >> >> On Wed, Feb 3, 2010 at 10:41 AM, Daniel Verkamp <daniel at drv.nu> wrote:
>> > [...]
>> >> >> + ? ?c->pic.buffer_hints = FF_BUFFER_HINTS_VALID | FF_BUFFER_HINTS_PRESERVE
>> >> >> + ? ? ? ? ? ? ? ? ? ? ? ?| FF_BUFFER_HINTS_REUSABLE;
>> >> >
>> >> > that should have no effect if you dont use *get_buffer()
>> >> >
>> >>
>> >> Ok, removed.
>> > [...]
>> >> + ? ?c->pic.buffer_hints = FF_BUFFER_HINTS_VALID | FF_BUFFER_HINTS_PRESERVE
>> >> + ? ? ? ? ? ? ? ? ? ? ? ?| FF_BUFFER_HINTS_REUSABLE;
>> >
>> > the more certain people are that they attached a patch
>> > the less certain it is the correct one
>> >
>>
>> Indeed, git user error... once more, with feeling.
>
> [...]
>
>> + ? ? ? ? ? ? ? ?if (outcnt - offset < 0)
>
> if (outcnt < offset)
>
> should be ok except that

Committed with change, thanks.

-- Daniel Verkamp



More information about the ffmpeg-devel mailing list