[MPlayer-dev-eng] [PATCH] Try to adapt to recent compilers such as GCC 14

Alexander Strasser eclipse7 at gmx.net
Tue Dec 24 17:31:41 EET 2024


Hi komh,
hi Ingo,
hi all!

On 2024-10-02 21:09 +0900, KO Myung-Hun wrote:
> Ingo Brückl wrote:
> > Lorenzo wrote on Tue, 1 Oct 2024 14:25:54 +0200:
> >
> >> On Tue, 01 Oct 2024 14:13:36 +0200
> >> Ingo Brückl <ib at oddnet.de> wrote:
> >
> >>> Lorenzo wrote on Tue, 1 Oct 2024 12:52:48 +0200:
> >>>
> >>>> with this patch applied it still fails[1] to build on i386 (Debian)
> >>>> for me, with
> >>>
> >>>>    73 |     if (iconv(inv_msgiconv, (const char **)&filename, &filename_len,
> >>>>       |                             ^~~~~~~~~~~~~~~~~~~~~~~~
> >>>>       |                             |
> >>>>       |                             const char **
> >>>
> >>> Which patch? It's okay in current svn.
> >
> >> Apologies, I was not subscribed so I just copied the subject without a
> >> proper quote; the patch posted in
> >> https://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2024-September/074217.html
> >
> > This patch is broken regarding all (const char **) casts for iconv()
> > which takes char ** as a second argument.
> >
>
> Hmm... OS/2 iconv() has `const' qualifier.
>
> Anyway I've attached the fixed patch.

Most of the patch looks OK, besides the iconv stuff, that I need to
look into more...

@komh:
Did you try to only fix warnings that became errors recently?
Or did you aim for also reducing some more warnings?


  Alexander


More information about the MPlayer-dev-eng mailing list