[vlc-devel] [PATCH] ccda: fix compiler warning

[George Vdl]

Hi,

Thank you for your feedback, you are right! It's a typo, I wanted to write
10000 instead of 1000, will fix asap!

On Mon, Apr 12, 2021, 19:24 Tristan Matthews <le.businessman at gmail.com>
wrote:

> Hi,
>
> On Mon, Apr 12, 2021 at 12:14 PM George Vaindirlis
> <george.vdl1 at gmail.com> wrote:
> >
> > fix warning: ‘%u’ directive output may be truncated writing between 1
> and 10 bytes into a region of size 5 [-Wformat-truncation=]
> >
> > ---
> >  modules/access/cdda.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/modules/access/cdda.c b/modules/access/cdda.c
> > index 9260b833ed..bd82bbd27e 100644
> > --- a/modules/access/cdda.c
> > +++ b/modules/access/cdda.c
> > @@ -635,7 +635,7 @@ static void AccessGetMeta(stream_t *access,
> vlc_meta_t *meta)
> >              vlc_meta_SetGenre(meta, str);
> >
> >          const unsigned year = cddb_disc_get_year(sys->cddb);
> > -        if (year != 0)
> > +        if (year != 0 && year < 1000)
>
> Maybe I'm missing something but won't this break for all but early
> medieval music?
>
> Best,
> -t
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20210412/f13aa46d/attachment.html>