[vlc-devel] [Patch] invmem feature and bugfix

[Hugo Beauzee-Luyssen]

On Wed, Sep 16, 2009 at 10:59 PM, Christophe Mutricy <xtophe at chewa.net> wrote:
> On Wed, Sep 16, 09 at 17:34 +0200, Hugo Beauzee-Luyssen wrote:
>> 2009/9/16 Rémi Denis-Courmont <remi at remlab.net>:
>> > Doesn't that kinda breaks backward compatibility??
>> >
>>
>> Don't know, I just tested it on master branch, I know it solved
>> problems for me, without creating some I can spot.
>
>
> What Rémi means is that invmem exist in the current release. And as your
> 2 new options don't have default values, you'll break working existing
> command lines.
>
> It's trivial to have the chroma defaulting to the old hardcodec value.
>
> Not too sure for the pitch, is there a generic formula per chroma/group
> of chroma which would work in the majority of common cases?
>

Well I guess these patch should fix it :

Regards,


-- 
Hugo Beauzée-Luyssen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Adding-a-invmem-chroma-parameter.patch
Type: application/octet-stream
Size: 5381 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090917/306ff1c3/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Don-t-store-the-picture_t-inside-of-the-decoder_sys_.patch
Type: application/octet-stream
Size: 2363 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090917/306ff1c3/attachment-0001.obj>