Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Fernando Nasser <fnasser@cygnus.com>
To: Michael Snyder <msnyder@redhat.com>
Cc: Michael Snyder <msnyder@cygnus.com>, gdb-patches@sources.redhat.com
Subject: Re: [PATCH RFA] ending-run.exp
Date: Fri, 13 Jul 2001 12:47:00 -0000	[thread overview]
Message-ID: <3B4F5033.CE549EAA@cygnus.com> (raw)
In-Reply-To: <3B4E3B5F.55D3@redhat.com>

Michael Snyder wrote:
> 
> Fernando Nasser wrote:
> >
> > Michael Snyder wrote:
> > >
> > > This script seems to expect that, if you step past the return from main,
> > > you will end up in some sort of "start" function.
> > >
> > > Can anyone think of any reason why "Program exited normally" is not
> > > a legitimate output to see after stepping past the return from main?
> > >
> >
> > Sorry for the delay, I was on vacation.
> >
> > Well, main() is normally called from inside crt0  from some sort of "start" function.  When you step past the return from main you should return to the frame above main, i.e., to after the call to main inside the "start" function in crt0.
> >
> > If it is not stopping there it is either a bug or some new situation that did not use to exist.
> >
> > I don't think we should accept "Program exited normally" until we understand a little bit more of what is actually happening when we get this message.
> 
> The thing is, this is testing something outside of GDB.
> GDB doesn't have any control over how main() is called.
> I don't think "stopping in the caller of main" should be
> part of GDB's expected behavior set.  Besides, the test
> fails on native linux, so _something_ is obviously wrong
> with it...

Humm...  we already test stopping on the caller in other places, so I am inclined to agree with you that this does not matter much here.

Nobody else has presented another reason not to do this, so lets go ahead and check your changes in.

-- 
Fernando Nasser
Red Hat - Toronto                       E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9


  reply	other threads:[~2001-07-13 12:47 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-07-02 14:43 Michael Snyder
     [not found] ` <3B4A132E.AA2DE377@cygnus.com>
2001-07-09 13:47   ` Andrew Cagney
2001-07-12 17:09   ` Michael Snyder
2001-07-13 12:47     ` Fernando Nasser [this message]
2001-07-25 15:42       ` Michael Snyder
2001-07-02 15:18 Michael Elizabeth Chastain

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=3B4F5033.CE549EAA@cygnus.com \
    --to=fnasser@cygnus.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=msnyder@cygnus.com \
    --cc=msnyder@redhat.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