[vlc-devel] [PATCH] input/mrl_helpers: fix warning related to iteration variable
Filip Roséen
filip at atch.se
Mon Feb 20 18:14:04 CET 2017
Hi Rémi,
On 2017-02-20 18:32, Rémi Denis-Courmont wrote:
> Le maanantaina 20. helmikuuta 2017, 14.53.50 EET Filip Roséen a écrit :
> > Following the changes in 7ecd927 the changed line caused a warning
> > related to signed vs unsigned integer comparision; this simply silence
> > that warning by using the now relevant type.
>
> Well yes but I left it because this code should use init/clear instead of new/
> destroy (and I have not yet fixed that occurrence).
Understood, I just wanted to get rid of the current warnings as they
hurt my eyes. Unless you plan on fixing the migration from
`vlc_array_new` to `vlc_array_init` and `vlc_array_clear` tonight I'd
be happy to do it.
While we are speaking about the matter, do you think it would make
sense to introduce a *`vlc_array_t`-initializer* that could be used as
the below (not sure about the name though).
vlc_array_t foo = VLC_ARRAY_INIT;
instead of having:
vlc_array_t foo;
vlc_array_init( &foo );
Best Regards,\
Filip Roséen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170220/87c84703/attachment.html>
More information about the vlc-devel
mailing list