From: Michael Elizabeth Chastain <chastain@cygnus.com>
To: chastain@cygnus.com, msnyder@cygnus.com
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] testsuite/gdb.c++/cplusfuncs.{exp,cc}: work with either g++ demangler
Date: Mon, 12 Feb 2001 12:30:00 -0000 [thread overview]
Message-ID: <200102122030.MAA22848@bosch.cygnus.com> (raw)
Hi Michael,
> I'm curious why you didn't simply write separate regular expressions,
> one to recognize the new demangler style and one to recognize the old,
> rather than add all this new mechanics.
That's a reasonable question.
The script does not actually know about about "v2 demangler" and "v3
demangler" and which regexp's go with which version. I wrote it more
like autoconf (although I am not experienced with autoconf). It probes
for each type separately and accommodates all the versions that I
have ever seen. If tomorrow's demangler wants to print 'foo &',
or report every 'int' as a 'signed int', it would be easy to maintain
the script.
> I'm concerned about the tcl scripts becoming complicated and difficult
> to maintain.
There are two parts. The probing part is meant to be independent and
can be copied-and-pasted into other scripts or put into a library.
The probing part can also be reduced to nothing if we stopped supporting
v2 abi. The strings are initialized to their v3 values.
As far as the use goes -- it does bother me to fill up the whole script
with "$dm_type_foo_ref".
Part of the problem is that a lot of these test scripts are not very
careful about "input to gdb" (which must always be a literal string)
versus "output from gdb" (which they can model with either a literal
string or with a regular expression). cplusfuncs.exp likes to specify
one string for both input and output, so it *has* to be a literal.
Michael
next reply other threads:[~2001-02-12 12:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-02-12 12:30 Michael Elizabeth Chastain [this message]
-- strict thread matches above, loose matches on Subject: below --
2001-02-14 9:21 Michael Elizabeth Chastain
[not found] <200102050418.UAA29343@bosch.cygnus.com>
2001-02-14 9:00 ` Fernando Nasser
2001-02-11 15:49 Michael Elizabeth Chastain
2001-02-12 12:12 ` Michael Snyder
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200102122030.MAA22848@bosch.cygnus.com \
--to=chastain@cygnus.com \
--cc=gdb-patches@sources.redhat.com \
--cc=msnyder@cygnus.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox