This is the mail archive of the gdb-prs@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]

[Bug c++/8218] ptype claims destructors have arg


https://sourceware.org/bugzilla/show_bug.cgi?id=8218

--- Comment #5 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Keith Seitz <kseitz@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=5f4d10850850cd95af5e95a16848c8c07a273d88

commit 5f4d10850850cd95af5e95a16848c8c07a273d88
Author: Keith Seitz <keiths@redhat.com>
Date:   Fri Mar 10 10:32:09 2017 -0800

    c++/8218: Destructors w/arguments.

    For a long time now, c++/8218 has noted that GDB is printing argument types
    for destructors:

    (gdb) ptype A
    type = class A {
      public:
        ~A(int);
    }

    This happens because cp_type_print_method_args doesn't ignore artificial
    arguments.  [It ignores the first `this' pointer because it simply skips
    the first argument for any non-static function.]

    This patch fixes this:

    (gdb) ptype  A
    type = class A {
      public:
        ~A();
    }

    I've adjusted gdb.cp/templates.exp to account for this and added a new
    passing regexp.

    gdb/ChangeLog

        PR c++/8218
        * c-typeprint.c (cp_type_print_method_args): Skip artificial arguments.

    gdb/testsuite/ChangeLog

        PR c++/8128
        * gdb.cp/templates.exp (test_ptype_of_templates): Remove argument
        type from destructor regexps.
        Add a branch which actually passes the test.
        Adjust "ptype t5i" test names.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

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