Re: RFA: C++ test for PR 1210

Re: RFA: C++ test for PR 1210

[Michael Elizabeth Chastain]

Proofread lightly, not tested.  Approved.

Michael C
rising from his coffin ...

Re: RFA: C++ test for PR 1210

[Daniel Jacobowitz]

On Sun, Jun 22, 2003 at 11:11:51AM -0400, Michael Elizabeth Chastain wrote:
> Proofread lightly, not tested.  Approved.
> 
> Michael C
> rising from his coffin ...

Thanks - and good to have you back again.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

RFA: C++ test for PR 1210

[Daniel Jacobowitz]

I'm just about to check in a patch to fix this bug.  Testcase fails without
the patch, passes with.  Look OK?

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

2003-06-21  Daniel Jacobowitz  <drow@mvista.com>

	* gdb.c++/pr-1210.cc: New file.
	* gdb.c++/pr-1210.exp: New file.

--- /dev/null	1969-12-31 19:00:00.000000000 -0500
+++ testsuite/gdb.c++/pr-1210.cc	2003-06-21 19:11:18.000000000 -0400
@@ -0,0 +1,19 @@
+class A
+{
+};
+
+class B : virtual public A
+{
+};
+
+class C : public A
+{
+     protected:
+         B myB;
+};
+
+int main()
+{
+     C *obj = new C();
+     return 0;
+}
--- /dev/null	1969-12-31 19:00:00.000000000 -0500
+++ testsuite/gdb.c++/pr-1210.exp	2003-06-21 19:20:51.000000000 -0400
@@ -0,0 +1,76 @@
+# Copyright 2003 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+# 
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+# 
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  
+
+# Tests for PR gdb/1210.
+
+# This file is part of the gdb testsuite.
+
+if $tracelevel then {
+	strace $tracelevel
+}
+
+if { [skip_cplus_tests] } { continue }
+
+#
+# test running programs
+#
+set prms_id 0
+set bug_id 0
+
+set testfile "pr-1210"
+set srcfile ${testfile}.cc
+set binfile ${objdir}/${subdir}/${testfile}
+
+if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug c++}] != "" } {
+    gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
+}
+
+if [get_compiler_info ${binfile} "c++"] {
+    return -1
+}
+
+gdb_exit
+gdb_start
+gdb_reinitialize_dir $srcdir/$subdir
+gdb_load ${binfile}
+
+if ![runto_main] then {
+    perror "couldn't run to breakpoint"
+    continue
+}
+
+gdb_test "next" ".*return 0;" "step past initialization"
+
+gdb_test_multiple "print *obj" "" {
+  -re "Cannot access memory.*$gdb_prompt $" {
+    fail "print *obj"
+  }
+  -re " = {<A> = {<No data fields>}, myB = {<A> = {<No data fields>}.*}}\r\n$gdb_prompt $" {
+    pass "print *obj"
+  }
+}
+
+gdb_test_multiple "print obj->myB" "" {
+  -re "Cannot access memory.*$gdb_prompt $" {
+    fail "print obj->myB"
+  }
+  -re " = {<A> = {<No data fields>}.*}\r\n$gdb_prompt $" {
+    pass "print obj->myB"
+  }
+}
+
+gdb_exit
+return 0

Re: RFA: C++ test for PR 1210

[David Carlton]

On Sat, 21 Jun 2003 19:26:12 -0400, Daniel Jacobowitz <drow@mvista.com> said:

> I'm just about to check in a patch to fix this bug.  Testcase fails
> without the patch, passes with.  Look OK?

Could you go and add some copyright boilerplate to the top of the .cc
file?  From previous discussion, they should have the boilerplate
there as well; I'm not up for tracking down the history of existing
files, but I'd prefer at least that new files have the boilerplate on
them.

David Carlton
carlton@kealia.com