[PATCH] gdb: testsuite: show print array-indexes after set in arrayidx.exp

[PATCH] gdb: testsuite: show print array-indexes after set in arrayidx.exp

[Tiezhu Yang]

Add "show print array-indexes" testcases after set print array-indexes
to off or on.

Without this patch:

    PASS: gdb.base/arrayidx.exp: set print array-indexes to off
    PASS: gdb.base/arrayidx.exp: print array with array-indexes off
    PASS: gdb.base/arrayidx.exp: set print array-indexes to on
    PASS: gdb.base/arrayidx.exp: print array with array-indexes on

With this patch:

    PASS: gdb.base/arrayidx.exp: set print array-indexes to off
    PASS: gdb.base/arrayidx.exp: show print array-indexes is off
    PASS: gdb.base/arrayidx.exp: print array with array-indexes off
    PASS: gdb.base/arrayidx.exp: set print array-indexes to on
    PASS: gdb.base/arrayidx.exp: show print array-indexes is on
    PASS: gdb.base/arrayidx.exp: print array with array-indexes on

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 gdb/testsuite/gdb.base/arrayidx.exp | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/gdb/testsuite/gdb.base/arrayidx.exp b/gdb/testsuite/gdb.base/arrayidx.exp
index af792ed..b2d52cf 100644
--- a/gdb/testsuite/gdb.base/arrayidx.exp
+++ b/gdb/testsuite/gdb.base/arrayidx.exp
@@ -36,6 +36,10 @@ if ![runto_main] then {
 gdb_test_no_output "set print array-indexes off" \
          "set print array-indexes to off"
 
+gdb_test "show print array-indexes" \
+         "Printing of array indexes is off." \
+         "show print array-indexes is off"
+
 gdb_test "print array" \
          "\\{1, 2, 3, 4\\}" \
          "print array with array-indexes off"
@@ -45,6 +49,10 @@ gdb_test "print array" \
 gdb_test_no_output "set print array-indexes on" \
          "set print array-indexes to on"
 
+gdb_test "show print array-indexes" \
+         "Printing of array indexes is on." \
+         "show print array-indexes is on"
+
 gdb_test "print array" \
          "\\{\\\[0\\\] = 1, \\\[1\\\] = 2, \\\[2\\\] = 3, \\\[3\\\] = 4\\}" \
          "print array with array-indexes on"
-- 
2.1.0

Re: [PATCH] gdb: testsuite: show print array-indexes after set in arrayidx.exp

[Andrew Burgess]

* Tiezhu Yang <yangtiezhu@loongson.cn> [2021-11-24 15:13:07 +0800]:

> Add "show print array-indexes" testcases after set print array-indexes
> to off or on.
> 
> Without this patch:
> 
>     PASS: gdb.base/arrayidx.exp: set print array-indexes to off
>     PASS: gdb.base/arrayidx.exp: print array with array-indexes off
>     PASS: gdb.base/arrayidx.exp: set print array-indexes to on
>     PASS: gdb.base/arrayidx.exp: print array with array-indexes on
> 
> With this patch:
> 
>     PASS: gdb.base/arrayidx.exp: set print array-indexes to off
>     PASS: gdb.base/arrayidx.exp: show print array-indexes is off
>     PASS: gdb.base/arrayidx.exp: print array with array-indexes off
>     PASS: gdb.base/arrayidx.exp: set print array-indexes to on
>     PASS: gdb.base/arrayidx.exp: show print array-indexes is on
>     PASS: gdb.base/arrayidx.exp: print array with array-indexes on
> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  gdb/testsuite/gdb.base/arrayidx.exp | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/gdb/testsuite/gdb.base/arrayidx.exp b/gdb/testsuite/gdb.base/arrayidx.exp
> index af792ed..b2d52cf 100644
> --- a/gdb/testsuite/gdb.base/arrayidx.exp
> +++ b/gdb/testsuite/gdb.base/arrayidx.exp
> @@ -36,6 +36,10 @@ if ![runto_main] then {
>  gdb_test_no_output "set print array-indexes off" \
>           "set print array-indexes to off"
>  
> +gdb_test "show print array-indexes" \
> +         "Printing of array indexes is off." \

This should be:

         "Printing of array indexes is off\\." \

Otherwise the '.' is matching any character.

> +         "show print array-indexes is off"
> +
>  gdb_test "print array" \
>           "\\{1, 2, 3, 4\\}" \
>           "print array with array-indexes off"
> @@ -45,6 +49,10 @@ gdb_test "print array" \
>  gdb_test_no_output "set print array-indexes on" \
>           "set print array-indexes to on"
>  
> +gdb_test "show print array-indexes" \
> +         "Printing of array indexes is on." \

Same again here.

OK to apply with these two fixes.

Thanks,
Andrew


> +         "show print array-indexes is on"
> +
>  gdb_test "print array" \
>           "\\{\\\[0\\\] = 1, \\\[1\\\] = 2, \\\[2\\\] = 3, \\\[3\\\] = 4\\}" \
>           "print array with array-indexes on"
> -- 
> 2.1.0
>