Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Bob Rossi <bob@brasko.net>
To: Vladimir Prus <vladimir@codesourcery.com>
Cc: Andrew Burgess <aburgess@broadcom.com>, gdb@sourceware.org
Subject: Re: MI async status output
Date: Fri, 18 Apr 2014 13:11:00 -0000	[thread overview]
Message-ID: <20140418104619.GA26892@linux> (raw)
In-Reply-To: <5350E049.9070705@codesourcery.com>

On Fri, Apr 18, 2014 at 12:20:25PM +0400, Vladimir Prus wrote:
> On 12.04.2014 04:25, Bob Rossi wrote:
> 
> >3)
> >     +download,{section=".interp",section-size="28",total-size="2466"}
> >              ^ (, eaten here)
> >     async-output ==>
> >         async-class ( "," result )*
> >
> >4) Error here: result must start with a variable name.
> >     +download,{section=".interp",section-size="28",total-size="2466"}
> >               ^ (syntax error)
> >     result ==>
> >         variable "=" value
> >     variable ==>
> >         string
> >
> >Did I find a bug or is this well known behavior of GDB and MI?
> >
> >If it's well known behavior, can anyone explain it? It's possible that
> >result could be just a 'value' instead of 'variable = value'. In this
> >case that would solve the problem perhaps.
> 
> whereas MI has grammar, the fact that actual output does not always match the
> grammar is well known. This specific problem was not known to me.
> 
> It is obviously possible to fix in a parser. It's also possible to fix in GDB,
> but as usual the question of what existing frontends might depend on this behaviour.

Thanks for the response. I'm writing a new grammar that will be open
source that handles as many possible outputs that GDB outputs, for as
many possible GDB versions. I'm writing unit and system tests to
validate this effort.

I'm taking notes every time i have to modify the parser to detail the
reasons why.

When I'm done, perhaps we can update GDB's manual with the new grammar
that I constuct, considering the one in the manual is just plain wrong.

Thanks,
Bob Rossi


  reply	other threads:[~2014-04-18 10:46 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-09 21:08 Bob Rossi
2014-04-10 15:01 ` Vladimir Prus
2014-04-10 20:13   ` Vladimir Prus
2014-04-11 10:01   ` Bob Rossi
2014-04-11 12:57     ` Andrew Burgess
2014-04-12  0:25       ` Bob Rossi
2014-04-14  6:25       ` Bob Rossi
2014-04-17 12:07         ` Bob Rossi
2014-04-18 10:46         ` Vladimir Prus
2014-04-18 13:11           ` Bob Rossi [this message]
2014-04-18 16:30             ` Vladimir Prus
2014-04-18 16:42               ` Bob Rossi
2014-04-18 17:59                 ` Vladimir Prus
2014-04-18 19:27                   ` Bob Rossi
2014-04-18 20:39                   ` Terekhov, Mikhail
2014-04-19  8:28                     ` Bob Rossi

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=20140418104619.GA26892@linux \
    --to=bob@brasko.net \
    --cc=aburgess@broadcom.com \
    --cc=gdb@sourceware.org \
    --cc=vladimir@codesourcery.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