This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFA] Fix parameter coercing for Ada function calls (take 2)


Hi Daniel,

Does this look like what you had in mind? It's a little more work
than my first patch, but indeed a little more elegant.

2008-01-08  Joel Brobecker  <brobecker@adacore.com>

        * ada-lang.c (ada_convert_actual): Renames convert_actual.
        Make non-static.
        (ada_convert_actuals): Delete.
        * ada-lang.c (ada_convert_actual): Add declaration.
        (ada_convert_actuals): Remove declaration.
        * infcall.c: #include "ada-lang.h".
        (value_arg_coerce): Add new parameter sp.  Add handling of Ada
        function call parameters.
        * Makefile.in (infcall.o): Update dependencies.

I am also reposting the testcase for everyone's convenience
(I feel sufficiently confident about the testcase that's very simple
to auto-review myself, but a second pair of eyes is always appreciated).

2008-01-08  Joel Brobecker  <brobecker@adacore.com>

        * gdb.ada/arrayparam: New test program.
        * gdb.ada/arrayparam.exp: New testcase.

All tested on x86-linux, no regression.

OK to commit?

Thanks,
-- 
Joel

Attachment: arrayparam.diff
Description: Text document

Attachment: arrayparam-tc.diff
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]