Re: [PATCH] libgloss: Fix invalid prototype

Jeff Johnston <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <CAOox84v_PC6KxxBxPNTQf7OTtZLgQivoxqP105cjczkLTUAQuw@mail.gmail.com>
Patch pushed.

-- Jeff J.

On Mon, Jun 9, 2025 at 9:03 AM Paul Cercueil <[email protected]> wrote:

> The __inln prototype was defined as:
>
> extern char * __inln ();
>
> In old versions of C, this meant "this function takes an unspecified
> number of parameters". This seems to have changed, as GCC 14 now
> considers that the function took zero parameters.
>
> As the code called it with one parameter, GCC would then issue an error
> saying that the incorrect number of parameters was passed.
>
> Address this issue by changing the prototype of the __inln function.
>
> Signed-off-by: Paul Cercueil <[email protected]>
> ---
>  libgloss/rs6000/mvme-read.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libgloss/rs6000/mvme-read.c b/libgloss/rs6000/mvme-read.c
> index 1c823b4..9d1e7d3 100644
> --- a/libgloss/rs6000/mvme-read.c
> +++ b/libgloss/rs6000/mvme-read.c
> @@ -14,7 +14,7 @@
>   */
>
>  extern int inbyte ();
> -extern char * __inln ();
> +extern char * __inln (char *);
>
>  /*
>   * read  -- read bytes from the serial port. Ignore fd, since
> --
> 2.47.2
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.