[patch] more gdb.c++/templates.exp failures

This patch converts some more gdb.c++/templates.exp failures (seen
under GCC 3.1, DWARF 2).

"ptype T5<int>" and "ptype t5i" give the same output; they suffer from
gdb/1111 and gdb/1113.  I KFAILed them with respect to the former,
mentioning the latter in a comment.

"constructor breakpoint" triggers gdb/1062.

"destructor breakpoint" triggers gdb/1112.  I changed this from
gdb_test into send_gdb/gdb_expect instead of gdb_test_multiple.  I
realize this makes me a bad person; one of these days I'll play around
with gdb_test_multiple, honest I will.

"print Garply<Garply<char> >::garply" isn't a bug at all: as happened
with one of the classes.exp tests, GDB prints out a 'const' in a
situation where that's perfectly appropriate.

Tested on i686-pc-linux-gnu/GCC3.1/DWARF-2; on that platform, "runtest
gdb.c++/*.exp" prints the following happy output:

		=== gdb Summary ===

# of expected passes		1608
# of known failures		25
/extra/gdb/mirror/src/gdb/testsuite/../../gdb/gdb version 2003-03-03-cvs -nx

I'm planning to commit these tomorrow unless somebody objects.

David Carlton

Re: [rfa:doco] Fix that stabs warning ...

>> Date: Mon, 03 Mar 2003 14:18:28 -0500
>> From: Andrew Cagney <ac131313 <at> redhat.com>
>> 
>> The attatched patch fixes the makeinfo warning printed when generating
>> the stabs documentation:
>> 
>> makeinfo -o stabs.info=20
>> /home/scratch/PENDING/2003-03-03-doco-stabs-vars/src/gdb/doc/stabs.texinfo
>> /home/scratch/PENDING/2003-03-03-doco-stabs-vars/src/gdb/doc/stabs.texinfo:
>> 2051: warning: unlikely character , in  <at> var.
> 
> 
> Approved, thanks.

Thanks.  make info now gives no warnings!

Andrew

Re: [rfa:doco] document register_type()

>> Date: Sun, 02 Mar 2003 11:49:01 -0500
>> From: Andrew Cagney <ac131313 <at> redhat.com>
>> 
>> This adds doco for the register_type() architecture method.
>> 
>> ok?
> 
> 
> Okay with me, but please put REGISTER_VIRTUAL_TYPE in  <at> code.

Fixed/committed.  Thanks.

Andrew

Re: PATCH : H8300 Simulator File I/O Implementation [ping]

Venky,

Hopefully all of your patches are in the bug database.  I'll get to them 
(assuming that no other global maintainer does) once the file I/O is done.

Andrew

Re: [patch] more gdb.c++/templates.exp failures

On 03 Mar 2003 14:44:38 -0800, David Carlton <carlton <at> math.Stanford.EDU> said:

> "destructor breakpoint" triggers gdb/1112.  I changed this from
> gdb_test into send_gdb/gdb_expect instead of gdb_test_multiple.  I
> realize this makes me a bad person; one of these days I'll play around
> with gdb_test_multiple, honest I will.

Daniel's given me enough hand-holding so that I'll use
gdb_test_multiple (which, after all, really is easier to use than
send_gdb/gdb_expect); new version of the patch is below.

David Carlton
carlton <at> math.stanford.edu

2003-03-03  David Carlton  <carlton <at> math.stanford.edu>

	* gdb.c++/templates.exp (do_tests): Accept valid const in "print
	Garply<Garply<char> >:: garply".
	(test_ptype_of_templates): KFAIL "ptype T5<int>" and "ptype t5i"
	with respect to PR c++/1111; note also PR c++/1113.
	(test_template_breakpoints): KFAIL "constructor breakpoint" with
	respect to PR c++/1062.
	KFAIL "destructor breakpoint" with respect to PR c++/1112.

Index: templates.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.c++/templates.exp,v
retrieving revision 1.17
diff -u -p -r1.17 templates.exp
--- templates.exp	3 Mar 2003 18:39:47 -0000	1.17

Re: [RFA] ObjC Testsuite

Michael Elizabeth Chastain wrote:
> 
> On the next submission, please use 'cvs diff -N' to include the new
> files in the diff, rather than attaching a diff + a tarball.
> That's easier for me.
> 
> We need one or more maintainers for the new gdb.objc.  I volunteer to
> be one of the maintainers.  Adam, are you willing to be a maintainer?
> 

Ok. I'm willing.

> Also, this whole review process is getting messy, we are going to have
> several people pushing Adam in different directions.  I would like to
> figure out *first* who are going to be the maintainer(s) of
> gdb.objc, and then those people should be the reviewers of this patch.
> 

Here's an update. I changed the filenames to basicclass from myclass 
(perhaps not much better), added proc gdb_compile_objc and added 
linenumbers to the breakpoint tests so they are more specific.

2003-03-03  Adam Fedor  <fedor <at> gnu.org>

        * Makefile.in (ALL_SUBDIRS): Add gdb.objc
        * configure.in (AC_OUTPUT): Add gdb.objc/Makefile.
	* lib/gdb.exp (proc gdb_compile_objc): New procedure.

Re: Toplevel configure.in cleanup 2/n

On Feb 25, 2003, Nathanael Nerode <neroden <at> twcny.rr.com> wrote:

> 	* configure.in: Don't set enable_threads and enable_shared to 'no'.
> 	Remove bogus comments.  Remove redundant noconfigdirs.
> 	* configure: Regenerate.

What happens if no --enable/disable-threads flag is passed?
enable_threads would be set to no before, now it remains unset.  The
enable_shared default seems is no longer appropriate, since a number of
sub-directories actually default to shared libraries enabled.  Perhaps
we should SET_LIB_PATH unconditionally now?

--

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva <at> {redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva <at> {lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

Re: Toplevel configure.in cleanup 3/n

On Feb 25, 2003, Nathanael Nerode <neroden <at> twcny.rr.com> wrote:

> 	* configure.in: Replace ${libstdcxx_version} by its value.
> 	* configure: Regenerate.

Ok

--

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva <at> {redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva <at> {lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

Re: Toplevel configure.in cleanup 4/n

On Feb 25, 2003, Nathanael Nerode <neroden <at> twcny.rr.com> wrote:

> 	* configure.in: Don't set unused shell variables.  Normalize
> 	indentation.
> 	* configure: Regenerate.

Could I convince you to post a diff -b of this change, so that it's
easier to tell where content actually changes?  In fact, I'd argue
that reindentation should be a separate patch from removal of unused
variables.

--

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva <at> {redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva <at> {lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

Re: Toplevel configure.in cleanup 5/n

On Feb 25, 2003, Nathanael Nerode <neroden <at> twcny.rr.com> wrote:

> mh-dgux only sets RANLIB=true.  Which is redundant now.

Erhm.  This might be a work-around for a broken RANLIB, as opposed to
a missing one, but I somehow doubt this is not the case.  Ok.

> (config)
> 	* mh-dgux: Delete.
> (toplev)
> 	* configure.in: Remove reference to mh-dgux.
> 	* configure: Rebuild.

--

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva <at> {redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva <at> {lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer