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

Petr Sorfa wrote:
> 
> 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.

Sounds like a good plan.
 
> 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:13 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
2002-04-30 12:13   ` Michael Snyder [this message]
  -- 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=3CCEE9EB.AADF0940@redhat.com \
    --to=msnyder@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mec@shout.net \
    --cc=petrs@caldera.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