This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] Allow "disassemble 'Foo::bar(char *)'"

From: Eli Zaretskii <eliz at gnu dot org>
To: Paul Pluzhnikov <ppluzhnikov at google dot com>
Cc: tromey at redhat dot com, schwab at linux-m68k dot org, gdb-patches at sourceware dot org
Date: Sun, 22 Nov 2009 06:27:23 +0200
Subject: Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
References: <20091114001022.A0A0776CDD@ppluzhnikov.mtv.corp.google.com> <8ac60eac0911150916k70ac0f62hf349b0bc21aa58b7@mail.gmail.com> <m3my2mtjhp.fsf@fleche.redhat.com> <8ac60eac0911161028of5f2352lf1857db7a46a94b2@mail.gmail.com> <8ac60eac0911191539w4245739ka6405ba072b79395@mail.gmail.com> <m3pr7d9ket.fsf@fleche.redhat.com> <8ac60eac0911201642y138489ffvab73550bfa72a89d@mail.gmail.com> <20091121005931.GA24143@caradoc.them.org> <8ac60eac0911201708k2c9bc062w9c34765e03b83566@mail.gmail.com> <83lji0p96p.fsf@gnu.org> <8ac60eac0911211838g44519802t5f082446d5494f2d@mail.gmail.com>
Reply-to: Eli Zaretskii <eliz at gnu dot org>

> Date: Sat, 21 Nov 2009 18:38:52 -0800
> From: Paul Pluzhnikov <ppluzhnikov@google.com>
> Cc: tromey@redhat.com, schwab@linux-m68k.org, gdb-patches@sourceware.org
> 
> > Is my interpretation of the test correct?  If it is, then the
> > documentation should say that whitespace is allowed around the comma.
> 
> The comma separates expressions, and expressions may contain
> {leading,trailing,in the middle} whitespace.
> 
> I've added a statement to gdb.texinfo to clarify that.

I still think that the whitespace should be explicitly mentioned.

Follow-Ups:
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov

References:
- [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Tom Tromey
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Tom Tromey
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Daniel Jacobowitz
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Eli Zaretskii
- Re: [patch] Allow "disassemble 'Foo::bar(char *)'"
  - From: Paul Pluzhnikov

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]