Re: [COMMITTED PATCH] Fix clang compilation errors in gdb.mi/basics.c

[Andrew Pinski <pinskia@gmail.com>]

On Mon, Aug 25, 2014 at 12:44 PM, Doug Evans <dje@google.com> wrote:
> Hi.
>
> This patch gets a few more tests working with clang.


Seems like maybe we should be compiling with -w for clang instead of
trying to change the testcase itself.  Because now we don't know if
the change to the testcase changed the behavior it was testing or not.

Thanks,
Andrew

>
> gdb compile failed, ../../../binutils-gdb/gdb/testsuite/gdb.mi/basics.c:32:1: warning: type
>       specifier missing, defaults to 'int' [-Wimplicit-int]
> callee3 (char *strarg)
> ^
> ../../../binutils-gdb/gdb/testsuite/gdb.mi/basics.c:35:1: warning: control
>       reaches end of non-void function [-Wreturn-type]
> }
> ^
> ...
>
> 2014-08-25  Doug Evans  <dje@google.com>
>
>         * gdb.mi/basics.c (callee3, callee2, callee1): Specify result type.
>         (main): Ditto.
>
> diff --git a/gdb/testsuite/gdb.mi/basics.c b/gdb/testsuite/gdb.mi/basics.c
> index 0a4f9de..0eabcd1 100644
> --- a/gdb/testsuite/gdb.mi/basics.c
> +++ b/gdb/testsuite/gdb.mi/basics.c
> @@ -29,17 +29,17 @@ int callee4 (void)
>    C = A + B;
>    return 0;
>  }
> -callee3 (char *strarg)
> +void callee3 (char *strarg)
>  {
>    callee4 ();
>  }
>
> -callee2 (int intarg, char *strarg)
> +void callee2 (int intarg, char *strarg)
>  {
>    callee3 (strarg);
>  }
>
> -callee1 (int intarg, char *strarg, double fltarg)
> +void callee1 (int intarg, char *strarg, double fltarg)
>  {
>    callee2 (intarg, strarg);
>  }
> @@ -57,7 +57,7 @@ void do_nothing (void)
>  {
>  }
>
> -main ()
> +int main ()
>  {
>    callee1 (2, "A string argument.", 3.5);
>    callee1 (2, "A string argument.", 3.5);