[vlc-devel] [PATCH] demux: mp4: avoid -Wformat warning with intermediate variable

Alexandre Janniaux ajanni at videolabs.io
Tue May 5 09:45:18 CEST 2020


Hi,

I'm not sure of the format allowed for MP4 box paths but
since care was taken to have a fixed size format descriptor
to serialize the name I preferred keeping it.

François can probably assert whether %u would be correct or
not too.

Regards,
--
Alexandre Janniaux
Videolabs

On Mon, May 04, 2020 at 09:36:14PM +0300, Rémi Denis-Courmont wrote:
> Le maanantaina 4. toukokuuta 2020, 20.39.02 EEST Alexandre Janniaux a écrit :
> > ---
> >  modules/demux/mp4/attachments.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/modules/demux/mp4/attachments.c
> > b/modules/demux/mp4/attachments.c index 59db7bb22bf..8376dbad053 100644
> > --- a/modules/demux/mp4/attachments.c
> > +++ b/modules/demux/mp4/attachments.c
> > @@ -315,10 +315,10 @@ int MP4_GetAttachments( const MP4_Box_t *p_root,
> > input_attachment_t ***ppp_attac if ( asprintf( &psz_location, "pnot[%u]",
> > i_index - 1 ) > -1 ) {
> >                  char rgz_path[14];
> > +                uint16_t index = p_pnot->data.p_pnot->i_index - 1;
> >                  snprintf( rgz_path, 14,
> >                           "/%4.4s[%"PRIu16"]",
> > -                         (const char *) &p_pnot->data.p_pnot->i_type,
> > -                         p_pnot->data.p_pnot->i_index - 1 );
> > +                         (const char *) &p_pnot->data.p_pnot->i_type, index
> > ); const MP4_Box_t *p_pict = MP4_BoxGet( p_root, rgz_path ); if( p_pict )
> >                  {
>
> But why? You could just use %u directly here.
>
> --
> Реми Дёни-Курмон
> http://www.remlab.net/
>
>
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list