diff mbox series

gdb/testsuite: use foreach_with_prefix in gdb.guile/scm-ports.exp

Message ID 20230911021329.1059595-1-simon.marchi@polymtl.ca
State New
Headers
Series gdb/testsuite: use foreach_with_prefix in gdb.guile/scm-ports.exp |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gdb_check--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 warning Patch is already merged
linaro-tcwg-bot/tcwg_gdb_build--master-arm warning Patch is already merged
linaro-tcwg-bot/tcwg_gdb_check--master-arm warning Patch is already merged

Commit Message

Simon Marchi Sept. 11, 2023, 2:13 a.m. UTC 
  Simplify things a bit using foreach_with_prefix.  The only expected
change is in the naming of tests.

Change-Id: Icb5e55207e0209e0d44d9e7c16a2f5e11aa29017
---
 gdb/testsuite/gdb.guile/scm-ports.exp | 114 ++++++++++++--------------
 1 file changed, 54 insertions(+), 60 deletions(-)


base-commit: ebc76ef6e6017b3548207c45abf33ab6e8f9402d

Comments

Andrew Burgess Sept. 11, 2023, 9:48 a.m. UTC | #1 
Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:

> Simplify things a bit using foreach_with_prefix.  The only expected
> change is in the naming of tests.

LGTM.

Approved-By: Andrew Burgess <aburgess@redhat.com>

Thanks,
Andrew

>
> Change-Id: Icb5e55207e0209e0d44d9e7c16a2f5e11aa29017
> ---
>  gdb/testsuite/gdb.guile/scm-ports.exp | 114 ++++++++++++--------------
>  1 file changed, 54 insertions(+), 60 deletions(-)
>
> diff --git a/gdb/testsuite/gdb.guile/scm-ports.exp b/gdb/testsuite/gdb.guile/scm-ports.exp
> index f0af5d4bbad2..7422a37345e5 100644
> --- a/gdb/testsuite/gdb.guile/scm-ports.exp
> +++ b/gdb/testsuite/gdb.guile/scm-ports.exp
> @@ -83,73 +83,67 @@ foreach variation $port_variations {
>  
>  # Test read/write of memory ports.
>  
> -proc test_mem_port_rw { kind } {
> -    if { "$kind" == "buffered" } {
> -	set buffered 1
> +proc test_mem_port_rw { buffered } {
> +    if $buffered {
> +	set mode "r+"
>      } else {
> -	set buffered 0
> +	set mode "r+0"
>      }
> -    with_test_prefix $kind {
> -	if $buffered {
> -	    set mode "r+"
> -	} else {
> -	    set mode "r+0"
> -	}
> -	gdb_test_no_output "guile (define rw-mem-port (open-memory #:mode \"$mode\"))" \
> -	    "create r/w memory port"
> -	gdb_test "guile (print rw-mem-port)" \
> -	    "#<input-output: gdb:memory-port 0x0-0xf+>"
> -	gdb_test_no_output "guile (define sp-reg (parse-and-eval \"\$sp\"))" \
> -	    "get sp reg"
> -	# Note: Only use $sp_reg for gdb_test result matching, don't use it in
> -	# gdb commands.  Otherwise transcript.N becomes unusable.
> -	set sp_reg [get_valueof /u "\$sp" 0]
> -	gdb_test_no_output "guile (define byte-at-sp (parse-and-eval \"*(char*) \$sp\"))" \
> -	    "save current value at sp"
> -	# Pass the result of parse-and-eval through value-fetch-lazy!,
> -	# otherwise the value gets left as a lazy reference to memory, which
> -	# when re-evaluated after we flush the write will yield the newly
> -	# written value.  PR 18175
> -	gdb_test_no_output "guile (value-fetch-lazy! byte-at-sp)" \
> -	    "un-lazyify byte-at-sp"
> -	gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
> -	    "= $sp_reg" \
> -	    "seek to \$sp"
> -	gdb_test_no_output "guile (define old-value (value->integer byte-at-sp))" \
> -	    "define old-value"
> -	gdb_test_no_output "guile (define new-value (logxor old-value 1))" \
> -	    "define new-value"
> -	gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 new-value)))" \
> -	    "= #<unspecified>"
> -	if $buffered {
> -	    # Value shouldn't be in memory yet.
> -	    gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
> -		"= #t" \
> -		"test byte at sp, before flush"
> -	    gdb_test_no_output "guile (force-output rw-mem-port)" \
> -		"flush port"
> -	}
> -	# Value should be in memory now.
> -	gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
> -	    "= #f" \
> -	    "test byte at sp, after flush"
> -	# Restore the value for cleanliness sake, and to verify close-port
> -	# flushes the buffer.
> -	gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
> -	    "= $sp_reg" \
> -	    "seek to \$sp for restore"
> -	gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 old-value)))" \
> -	    "= #<unspecified>"
> -	gdb_test "guile (print (close-port rw-mem-port))" \
> -	    "= #t"
> +    gdb_test_no_output "guile (define rw-mem-port (open-memory #:mode \"$mode\"))" \
> +	"create r/w memory port"
> +    gdb_test "guile (print rw-mem-port)" \
> +	"#<input-output: gdb:memory-port 0x0-0xf+>"
> +    gdb_test_no_output "guile (define sp-reg (parse-and-eval \"\$sp\"))" \
> +	"get sp reg"
> +    # Note: Only use $sp_reg for gdb_test result matching, don't use it in
> +    # gdb commands.  Otherwise transcript.N becomes unusable.
> +    set sp_reg [get_valueof /u "\$sp" 0]
> +    gdb_test_no_output "guile (define byte-at-sp (parse-and-eval \"*(char*) \$sp\"))" \
> +	"save current value at sp"
> +    # Pass the result of parse-and-eval through value-fetch-lazy!,
> +    # otherwise the value gets left as a lazy reference to memory, which
> +    # when re-evaluated after we flush the write will yield the newly
> +    # written value.  PR 18175
> +    gdb_test_no_output "guile (value-fetch-lazy! byte-at-sp)" \
> +	"un-lazyify byte-at-sp"
> +    gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
> +	"= $sp_reg" \
> +	"seek to \$sp"
> +    gdb_test_no_output "guile (define old-value (value->integer byte-at-sp))" \
> +	"define old-value"
> +    gdb_test_no_output "guile (define new-value (logxor old-value 1))" \
> +	"define new-value"
> +    gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 new-value)))" \
> +	"= #<unspecified>"
> +    if $buffered {
> +	# Value shouldn't be in memory yet.
>  	gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
>  	    "= #t" \
> -	    "test byte at sp, after close"
> +	    "test byte at sp, before flush"
> +	gdb_test_no_output "guile (force-output rw-mem-port)" \
> +	    "flush port"
>      }
> +    # Value should be in memory now.
> +    gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
> +	"= #f" \
> +	"test byte at sp, after flush"
> +    # Restore the value for cleanliness sake, and to verify close-port
> +    # flushes the buffer.
> +    gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
> +	"= $sp_reg" \
> +	"seek to \$sp for restore"
> +    gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 old-value)))" \
> +	"= #<unspecified>"
> +    gdb_test "guile (print (close-port rw-mem-port))" \
> +	"= #t"
> +    gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
> +	"= #t" \
> +	"test byte at sp, after close"
>  }
>  
> -test_mem_port_rw buffered
> -test_mem_port_rw unbuffered
> +foreach_with_prefix buffered {1 0} {
> +    test_mem_port_rw $buffered
> +}
>  
>  # Test zero-length memory ports.
>  
>
> base-commit: ebc76ef6e6017b3548207c45abf33ab6e8f9402d
> -- 
> 2.42.0
Simon Marchi Sept. 11, 2023, 4:32 p.m. UTC | #2 
On 9/11/23 05:48, Andrew Burgess wrote:
> Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:
> 
>> Simplify things a bit using foreach_with_prefix.  The only expected
>> change is in the naming of tests.
> 
> LGTM.
> 
> Approved-By: Andrew Burgess <aburgess@redhat.com>
> 
> Thanks,
> Andrew

Thanks, pushed.

Simon
diff mbox series

Patch

diff --git a/gdb/testsuite/gdb.guile/scm-ports.exp b/gdb/testsuite/gdb.guile/scm-ports.exp
index f0af5d4bbad2..7422a37345e5 100644
--- a/gdb/testsuite/gdb.guile/scm-ports.exp
+++ b/gdb/testsuite/gdb.guile/scm-ports.exp
@@ -83,73 +83,67 @@  foreach variation $port_variations {
 
 # Test read/write of memory ports.
 
-proc test_mem_port_rw { kind } {
-    if { "$kind" == "buffered" } {
-	set buffered 1
+proc test_mem_port_rw { buffered } {
+    if $buffered {
+	set mode "r+"
     } else {
-	set buffered 0
+	set mode "r+0"
     }
-    with_test_prefix $kind {
-	if $buffered {
-	    set mode "r+"
-	} else {
-	    set mode "r+0"
-	}
-	gdb_test_no_output "guile (define rw-mem-port (open-memory #:mode \"$mode\"))" \
-	    "create r/w memory port"
-	gdb_test "guile (print rw-mem-port)" \
-	    "#<input-output: gdb:memory-port 0x0-0xf+>"
-	gdb_test_no_output "guile (define sp-reg (parse-and-eval \"\$sp\"))" \
-	    "get sp reg"
-	# Note: Only use $sp_reg for gdb_test result matching, don't use it in
-	# gdb commands.  Otherwise transcript.N becomes unusable.
-	set sp_reg [get_valueof /u "\$sp" 0]
-	gdb_test_no_output "guile (define byte-at-sp (parse-and-eval \"*(char*) \$sp\"))" \
-	    "save current value at sp"
-	# Pass the result of parse-and-eval through value-fetch-lazy!,
-	# otherwise the value gets left as a lazy reference to memory, which
-	# when re-evaluated after we flush the write will yield the newly
-	# written value.  PR 18175
-	gdb_test_no_output "guile (value-fetch-lazy! byte-at-sp)" \
-	    "un-lazyify byte-at-sp"
-	gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
-	    "= $sp_reg" \
-	    "seek to \$sp"
-	gdb_test_no_output "guile (define old-value (value->integer byte-at-sp))" \
-	    "define old-value"
-	gdb_test_no_output "guile (define new-value (logxor old-value 1))" \
-	    "define new-value"
-	gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 new-value)))" \
-	    "= #<unspecified>"
-	if $buffered {
-	    # Value shouldn't be in memory yet.
-	    gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
-		"= #t" \
-		"test byte at sp, before flush"
-	    gdb_test_no_output "guile (force-output rw-mem-port)" \
-		"flush port"
-	}
-	# Value should be in memory now.
-	gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
-	    "= #f" \
-	    "test byte at sp, after flush"
-	# Restore the value for cleanliness sake, and to verify close-port
-	# flushes the buffer.
-	gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
-	    "= $sp_reg" \
-	    "seek to \$sp for restore"
-	gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 old-value)))" \
-	    "= #<unspecified>"
-	gdb_test "guile (print (close-port rw-mem-port))" \
-	    "= #t"
+    gdb_test_no_output "guile (define rw-mem-port (open-memory #:mode \"$mode\"))" \
+	"create r/w memory port"
+    gdb_test "guile (print rw-mem-port)" \
+	"#<input-output: gdb:memory-port 0x0-0xf+>"
+    gdb_test_no_output "guile (define sp-reg (parse-and-eval \"\$sp\"))" \
+	"get sp reg"
+    # Note: Only use $sp_reg for gdb_test result matching, don't use it in
+    # gdb commands.  Otherwise transcript.N becomes unusable.
+    set sp_reg [get_valueof /u "\$sp" 0]
+    gdb_test_no_output "guile (define byte-at-sp (parse-and-eval \"*(char*) \$sp\"))" \
+	"save current value at sp"
+    # Pass the result of parse-and-eval through value-fetch-lazy!,
+    # otherwise the value gets left as a lazy reference to memory, which
+    # when re-evaluated after we flush the write will yield the newly
+    # written value.  PR 18175
+    gdb_test_no_output "guile (value-fetch-lazy! byte-at-sp)" \
+	"un-lazyify byte-at-sp"
+    gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
+	"= $sp_reg" \
+	"seek to \$sp"
+    gdb_test_no_output "guile (define old-value (value->integer byte-at-sp))" \
+	"define old-value"
+    gdb_test_no_output "guile (define new-value (logxor old-value 1))" \
+	"define new-value"
+    gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 new-value)))" \
+	"= #<unspecified>"
+    if $buffered {
+	# Value shouldn't be in memory yet.
 	gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
 	    "= #t" \
-	    "test byte at sp, after close"
+	    "test byte at sp, before flush"
+	gdb_test_no_output "guile (force-output rw-mem-port)" \
+	    "flush port"
     }
+    # Value should be in memory now.
+    gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
+	"= #f" \
+	"test byte at sp, after flush"
+    # Restore the value for cleanliness sake, and to verify close-port
+    # flushes the buffer.
+    gdb_test "guile (print (seek rw-mem-port (value->integer sp-reg) SEEK_SET))" \
+	"= $sp_reg" \
+	"seek to \$sp for restore"
+    gdb_test "guile (print (put-bytevector rw-mem-port (make-bytevector 1 old-value)))" \
+	"= #<unspecified>"
+    gdb_test "guile (print (close-port rw-mem-port))" \
+	"= #t"
+    gdb_test "guile (print (value=? (parse-and-eval \"*(char*) \$sp\") byte-at-sp))" \
+	"= #t" \
+	"test byte at sp, after close"
 }
 
-test_mem_port_rw buffered
-test_mem_port_rw unbuffered
+foreach_with_prefix buffered {1 0} {
+    test_mem_port_rw $buffered
+}
 
 # Test zero-length memory ports.