From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stan Shebs <shebs@cygnus.com>
To: cagney@cygnus.com
Cc: gdb@cygnus.com
Subject: Re: More gdb-4.17.85 testresults
Date: Thu, 01 Apr 1999 00:00:00 -0000
Message-id: <199903100246.SAA12486@andros.cygnus.com>
References: <36E4A175.DEA4FA72@cygnus.com>
X-SW-Source: 1999-q1/msg00109.html

   Date: Tue, 09 Mar 1999 15:20:06 +1100
   From: Andrew Cagney <ac131313@cygnus.com>

   > cc: "../../gdb-4.17.85/gdb/remote.c", line 3046: \
   >     error 1711: Inconsistent parameter list declaration for "remote_query".
   >

       static int remote_query PARAMS ((char, char *, char *, int *));

   and:

       static int
       remote_query (query_type, buf, outbuf, bufsiz)
	    char query_type;
	    char *buf;
	    char *outbuf;
	    int *bufsiz;

   This time I've no idea's as to what HP's ansic doesn't like about it.  Any
   suggestions?

The actual problem is that the function definition is "K&R style" not
true ANSI, but in any case the right solution is to change the first
arg from "char" to "int".  While there may be situations where you
really need an argument to be declared "char", in 20 years of C
programming I've only seen one case (certain Mac Toolbox calls) where
having a char arg was important, and dozens of cases where it just
made problems for no good reason.

(This one is a little embarassing, because I went over the remote_query
code, and missed it...)

								Stan