[vlc-devel] smem: Use size_t for buffer sizes & fix warning

Hugo Beauzée-Luyssen hugo at beauzee.fr
Thu Jan 30 21:59:15 CET 2014


On Wed, Jan 29, 2014 at 7:32 PM, Hugo Beauzée-Luyssen <hugo at beauzee.fr> wrote:
> On Wed, Jan 29, 2014 at 10:32 AM, Rafaël Carré <funman at videolan.org> wrote:
>> On 01/28/14 23:42, Hugo Beauzée-Luyssen wrote:
>>> On Wed, Jan 29, 2014 at 12:35 AM, Rémi Denis-Courmont <remi at remlab.net> wrote:
>>
>>>> Debatable. What if another compiler version now complains that the value is
>>>> assigned but never used?
>>
>>
>>> Agreed for point 1 (including the part about strings). I guess I can
>>> also update the wiki.
>>> Point 2: it changes so little that I really don't care, up to the
>>> reviewers. Does not warn on gcc4.8, didn't try clang.
>>
>> If you initialize it to i_height and reuse i_line in
>> post_render_callback it will be alright
>>
>>> Regards,
>
> I'd argue that this is making the code less readable and is definitely
> not worth doing for the sake of a (wrong) warning.
>

If no-one objects I'll push the 1st patch, and keep the 2nd one for a
future trol^Wdiscussion.

Regards,

-- 
Hugo Beauzée-Luyssen
mail: hugo at beauzee.fr
skype: beauze.h



More information about the vlc-devel mailing list