[vlc-devel] [Patch] Solving an unused variable problem

Hugo Beauzee-Luyssen beauze.h at gmail.com
Thu Aug 20 18:40:04 CEST 2009


On Thu, Aug 20, 2009 at 6:37 PM, Pierre
d'Herbemont<pdherbemont at gmail.com> wrote:
> Hi Hugo,
>
> I have committed a slightly different fix. Thanks for reporting!
>
> Pierre.
>
> On Aug 20, 2009, at 6:28 PM, Hugo Beauzee-Luyssen wrote:
>
>> Hi all.
>>
>> In OSD module, in osd_menu.h, a ppsz_button_states is declared.
>> When parser.c includes this file, it doesn't use this variable, which
>> makes gcc angry, especially with the new Werror setting :p
>>
>> This patch solves it, in a "not to nasty way", since the only other
>> way i can think of is to have a single header file with the variable
>> declaration/definition in it...
>>
>> Regards,
>>
>> -- Hugo Beauzée-Luyssen
>>
>> <0002-Solved-unused-variable-problem.patch>_______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> http://mailman.videolan.org/listinfo/vlc-devel
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>

Hmmm yeah maybe I should have think of extern keyword :p
Seems more tricky :D

-- 
Hugo Beauzée-Luyssen



More information about the vlc-devel mailing list