Re: [PATCH] [gdb/exp] Fix cast handling for indirection

[Tom Tromey <tom@tromey.com>]

>>>>> "Kevin" == Kevin Buettner <kevinb@redhat.com> writes:

>> char *
>> a_loc (void)

Kevin> This surprised me.  I would have thought that the evaluation would
Kevin> have been:

Kevin>     (char)*(int)a_loc ()

Kevin> ...due to the fact that functions lacking an explicit return type
Kevin> return 'int' in traditional C.

Here the function actually returns char*, it's just the debug info is
missing.

Normally users should have written:

    print *(char *) a_loc()

here, at least after the changes a while ago to require a cast of the
return type here.  Longer ago, debuginfo-less functions did default to
int return in gdb, but Pedro (I think) changed this a while back.

I was wondering if this patch causes gdb to accept some weird things
that might have been rejected in the past, by introducing a hidden cast.
Maybe print (char) *85732 does something surprising now.  I'm not
entirely sure if that's bad.

Tom