Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Petr Sorfa <petrs@caldera.com>
To: Michael Elizabeth Chastain <mec@shout.net>
Cc: msnyder@redhat.com, gdb-patches@sources.redhat.com, shebs@apple.com
Subject: Re: [RFC] FORTRAN95 Expression parser
Date: Tue, 30 Apr 2002 12:01:00 -0000	[thread overview]
Message-ID: <3CCEEC82.3B6A1298@caldera.com> (raw)
In-Reply-To: <200204301810.g3UIAuE13676@duracef.shout.net>

Hi All,

Thanks for all the input and comments.

The side-by-side integration was my original intention, so that once
everybody realizes how great the new F95 parser is then a quick switch
can be done. Note that I'm thinking of automatically enabling it for
F90/F95 binaries (as long as they are appropriately marked by the DWARF
language attribute.) To do this I'll add a new language to gdb -
fortran95

Since the parser full functionality depends on many features touching
DWARF support, column major support, variable evaluation, location
expression, new scoping, etc.. which will require separate code reviews
and comments. I'll initially release the parser as a "crimped" version.
Basically it will do the parsing, but will raise an error if any of the
unsupported features are used. I'll be adding the other features
separately and enable the various functionality of the parser as it goes
on.

I'm hoping to roll out everything over the next couple of weeks. The
test suite update will be put in early as well.

Even with this roll-out plan I personally think that the initial
"crimped" version will be more robust than the existing FORTRAN parser.

Petr

> Coincidentally, I was looking at a FORTRAN pr this morning.
> 
> I think the existing FORTRAN expression parser is lame and I would
> not miss it.
> 
> I favor the side-by-side strategy.  Keep the files as f95-x files.
> Let the gdb people play with it side by side.  If the new interface
> dominates the old interface, we can remove the old f-x files.
> If we think there will be a constituency for the old interface,
> then we can release both.
> 
> The problem with merging is that it makes the code much less readable
> and makes it harder to discard the old code.
> 
> My two cents,
> 
> Michael C


  reply	other threads:[~2002-04-30 19:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-04-30 11:11 Michael Elizabeth Chastain
2002-04-30 12:01 ` Petr Sorfa [this message]
2002-04-30 12:13   ` Michael Snyder
  -- strict thread matches above, loose matches on Subject: below --
2002-04-29 11:57 Petr Sorfa
2002-04-29 18:49 ` Michael Snyder
2002-04-30  6:40   ` Petr Sorfa
2002-04-30 10:50     ` Michael Snyder
2002-04-30 11:02     ` Stan Shebs

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=3CCEEC82.3B6A1298@caldera.com \
    --to=petrs@caldera.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mec@shout.net \
    --cc=msnyder@redhat.com \
    --cc=shebs@apple.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