[vlc-devel] [PATCH] vaapi: fix profiles detection.

Jean-Paul Saman jpsaman at videolan.org
Tue Dec 6 15:50:26 CET 2011


On Tue, Dec 6, 2011 at 3:43 PM, Francois Cartegnie <fcvlcdev at free.fr> wrote:
> On 06/12/2011 15:16, Jean-Paul Saman wrote:
>> ==> cause of crash, it should be vaMaxNumProfiles( p_va->p_display );
>
> Of course that's wrong.
> Scrolled through source and ended copying 3 lines lower :/
>
>>> +        while( --i_profiles_nb >= 0 )
>>
>> The coding style in this file uses for-loops in this kind of
>> constructs. Therefor my suggestion is that you do the same.
>
> I tried to avoid another top declaration as I can't see any inline var
> decl here.

for( int i = 0 ; i < i_profiles_nb; i++ )
is allowed in VLC coding style.

Kind regards,
Jean-Paul Saman



More information about the vlc-devel mailing list