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: [RFA] Add test for use of "<block>::<variable>" syntax for locals in watch.


On 01/09/2012 03:26 PM, Paul Hilfinger wrote:
> 2012-01-07  Paul Hilfinger  <brobecker@adacore.com>
> 

Your mail address is not correct :)

I am not familiar with this test, but have some comments on format of
changelog.

> * gdb/testsuite/gdb.base/watchpoint.c (recurser): Initialize local_x.
>   (main) Repeat recurser call.
          ^^ Miss ":" here.
> * gdb/testsuite/gdb.base/watchpoint.exp: Check that 'watch recurser::local_x' is
>   equivalent to 'local_x'.

Each changelog entry is started from a tab.  Since we have a ChangeLog
file under gdb/testsuite, so we don't have to include "gdb/testuite" in
file name.  It can be like this,

	* gdb.base/watchpoint.c (recurser): Initialize local_x.
	(main): Repeat recurser call.
	* gdb.base/watchpoint.exp: Check that 'watch recurser::local_x' is
	equivalent to 'local_x'.

-- 
Yao (éå)


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