On 19/08/13 18:55, Vittorio Giovara wrote:
> Il giorno 18/ago/2013, alle ore 17:40, Jan Gerber <[email protected]> ha scritto:
> 
>> ---
>> libavformat/oggdec.c | 9 ++++++++-
>> 1 file changed, 8 insertions(+), 1 deletion(-)
>>
>>  + os->incomplete = !!os->psize; } } while (!complete);
> 
> I am not sure what the project coding style guidelines say about this,
> but I would really really prefer clarity over brevity and write
> 
>     os->incomplete = (os->psize != 0) ? 1 : 0;
> 
> if everyone agrees.

Personally I consider !! clear enough.

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to