Re: GDB/MI Output syntax

[Andrew Cagney <cagney@gnu.org>]

> Bob Rossi <bob@brasko.net> wrote:
> 
>>> -@code{@var{async-record} | @var{stream-record}}
>>> +@code{( @var{async-record} | @var{stream-record} ) @var{nl}}
> 
> 
> I'm dubious about this.
> 
> stream-record does not have an NL terminator and needs one, yes.
> 
> But async-record already gets an NL terminator in the grammar
> and does not need a second one.

That's kind of why, per my earlier post, I suggested moving all the 
@var{nl} to the @var{output} production.  That way we can see exactly 
where they fit in.

Andrew

>   async-record -> exec-async-output | status-async-output | notify-async-output
>   exec-async-output -> [token] "*" async-output
>   async-output -> async-class ("," result)* nl
> 
> For example, gdb.mi/gdb669.exp gives:
> 
>   # gdb.log excerpt
>   ~"[Thread debugging using libthread_db enabled]\n"
>   ~"[New Thread 8192 (LWP 26606)]\n"
>   ~"[Switching to Thread 8192 (LWP 26606)]\n"
>   000*stopped,reason="breakpoint-hit",bkptno="1",thread-id="1",frame={addr="0x0804852d",func="main",args=[{name="argc",value="1"},{name="argv",value="0xbffff814"}],file="/berman/fsf/_today_/source/gdb/HEAD/src/gdb/testsuite/gdb.mi/pthreads.c",line="81"}
>   (gdb) 
> 
>