This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFC] fullname attribute for GDB/MI stack frames
- From: Bob Rossi <bob at brasko dot net>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: Dennis Brueni <dbrueni at slickedit dot com>, gdb-patches at sources dot redhat dot com
- Date: Sun, 1 May 2005 20:54:15 -0400
- Subject: Re: [RFC] fullname attribute for GDB/MI stack frames
- References: <ECCC6E9907B4CD4A83260A191A91F20E22C6D2@wampa.office.slickedit.com> <20050430191755.GF7009@nevyn.them.org> <20050501021945.GA19962@white> <01c54e7a$Blat.v2.4$e31afae0@zahav.net.il>
On Sun, May 01, 2005 at 09:22:31PM +0300, Eli Zaretskii wrote:
> > Date: Sat, 30 Apr 2005 22:19:45 -0400
> > From: Bob Rossi <bob@brasko.net>
> >
> > On Sat, Apr 30, 2005 at 03:17:55PM -0400, Daniel Jacobowitz wrote:
> > > On Fri, Apr 01, 2005 at 02:13:33PM -0500, Dennis Brueni wrote:
> > > > > Again, for the fullname regex, I would recommend using the
> > > > > same regex used in mi-file.exp. This would be like
> > > > > fullname="/.*basics.c" This forces the regex to ensure that
> > > > > the path is absolute, which the check you have does not.
> > >
> > > Will GDB always output absolute paths that start with "/"? What about
> > > non-Cygwin Windows for example? DJGPP?
>
> To answer Daniel's question: no, absolute file names on DOS/Windows
> systems do not necessarily begin with a slash.
>
> > Like Dennis noted, it could be possible that the fullname might not
> > start with a "/". I originally posted the patch with the fullname
> > starting with a "/", and since then, there hasn't been any complaints.
> > If there is a better regex that ensures that the fullname is absolute
> > I'd be happy to change the mi-file.exp test to it.
>
> The proper regexp should be something like
>
> \([A-z]:\)?[/\\].*basics.c
Eli, is this correct? Is a proper fullname in windows \abc? Should the
regex move the \\ into the first grouping with the drive letter?
\([A-z]:\\\)?/.*basics.c
Please let me know.
Thanks,
Bob Rossi