public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [Patch, Testsuite] Update selftest.exp to match c source
@ 2010-08-17  6:19 Yao Qi
  2010-08-17  8:10 ` Joel Brobecker
  0 siblings, 1 reply; 3+ messages in thread
From: Yao Qi @ 2010-08-17  6:19 UTC (permalink / raw)
  To: gdb-patches

[-- Attachment #1: Type: text/plain, Size: 234 bytes --]

This patch is to update gdb.gdb/selftest.exp to reflect the latest
source file, otherwise, it will fail in some cases due to failing to
match source code.

Is that OK?

-- 
Yao Qi
CodeSourcery
yao@codesourcery.com
(650) 331-3385 x739

[-- Attachment #2: selftest.patch --]
[-- Type: text/x-diff, Size: 1213 bytes --]

gdb/testsuite/
2010-08-17  Yao Qi  <yao@codesourcery.com>

	* gdb.gdb/selftest.exp (do_steps_and_nexts): Update test case
	to reflect latest c source file.

Index: gdb/testsuite/gdb.gdb/selftest.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.gdb/selftest.exp,v
retrieving revision 1.24
diff -u -p -r1.24 selftest.exp
--- gdb/testsuite/gdb.gdb/selftest.exp	26 Jun 2010 06:44:47 -0000	1.24
+++ gdb/testsuite/gdb.gdb/selftest.exp	17 Aug 2010 05:58:37 -0000
@@ -185,11 +185,15 @@ proc do_steps_and_nexts {} {
 		set description "step over instream initialization"
 		set command "step"
 	    }
-	    -re ".*getcwd .gdb_dirbuf, sizeof .gdb_dirbuf..;.*$gdb_prompt $" {
+	    -re ".*getcwd .gdb_dirbuf, sizeof .gdb_dirbuf.*$gdb_prompt $" {
 		set description "next over getcwd"
 		set command "next"
 	    }
-	    -re ".*quit_flag = 0.*$gdb_prompt $" {
+	    -re ".*gdb_program_name = xstrdup.*$gdb_prompt $" {
+                set description "next over xstrdup"
+                set command "next"
+            } 
+            -re ".*quit_flag = 0.*$gdb_prompt $" {
 		set description "step over quit_flag initialization"
 		set command "step"
 	    }

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Patch, Testsuite] Update selftest.exp to match c source
  2010-08-17  6:19 [Patch, Testsuite] Update selftest.exp to match c source Yao Qi
@ 2010-08-17  8:10 ` Joel Brobecker
  2010-08-18  2:31   ` Yao Qi
  0 siblings, 1 reply; 3+ messages in thread
From: Joel Brobecker @ 2010-08-17  8:10 UTC (permalink / raw)
  To: Yao Qi; +Cc: gdb-patches

> gdb/testsuite/
> 2010-08-17  Yao Qi  <yao@codesourcery.com>
> 
> 	* gdb.gdb/selftest.exp (do_steps_and_nexts): Update test case
> 	to reflect latest c source file.

Thank you. This is OK.

It looks like some other statements might be missing, but we can worry
about those if/when we see failures.

-- 
Joel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Patch, Testsuite] Update selftest.exp to match c source
  2010-08-17  8:10 ` Joel Brobecker
@ 2010-08-18  2:31   ` Yao Qi
  0 siblings, 0 replies; 3+ messages in thread
From: Yao Qi @ 2010-08-18  2:31 UTC (permalink / raw)
  To: gdb-patches

On Tue, Aug 17, 2010 at 10:10:48AM +0200, Joel Brobecker wrote:
> > gdb/testsuite/
> > 2010-08-17  Yao Qi  <yao@codesourcery.com>
> > 
> > 	* gdb.gdb/selftest.exp (do_steps_and_nexts): Update test case
> > 	to reflect latest c source file.
> 
> Thank you. This is OK.
> 
> It looks like some other statements might be missing, but we can worry
> about those if/when we see failures.

Committed.
http://sourceware.org/ml/gdb-cvs/2010-08/msg00095.html

-- 
Yao Qi
CodeSourcery
yao@codesourcery.com
(650) 331-3385 x739

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-08-18  2:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-17  6:19 [Patch, Testsuite] Update selftest.exp to match c source Yao Qi
2010-08-17  8:10 ` Joel Brobecker
2010-08-18  2:31   ` Yao Qi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).