Re: [PATCH] Handle memory write errors on gdb.base/break-always.exp

[Yao Qi <qiyaoltc at gmail dot com>]

Luis Machado <lgustavo@codesourcery.com> writes:

Hi Luis,

> +foreach test_value {0 1} {
> +    set write_test "write $test_value to breakpoint's address $bp_address"
> +

In this way, $bp_address will be shown in the test summary in gdb.sum.
This may cause some differences when we compare gdb.sum of different
runs, like:

-PASS: gdb.base/break-always.exp: write 0 to breakpoint's address 0x1111
+PASS: gdb.base/break-always.exp: write 0 to breakpoint's address 0x2222

Let's remove $bp_address from $write_test.

> +    gdb_test_multiple "p /x *(char *) $bp_address = $test_value" $write_test {
> +	-re "Cannot access memory at address $hex.*$gdb_prompt $" {
> +	  unsupported "Cannot write to address $bp_address"

We need comments to explain we may go here.  Copying some messages from
your mail should be fine.

> +	  return -1
> +	}
> +	-re " = .*$gdb_prompt $" {
> +	  pass $write_test
> +	}
> +    }
> +
> +    set read_test "read back $test_value from the breakpoint's address $bp_address"
> +

Remove $bp_address from $read_test.

> +    gdb_test "p /x *(char *) $bp_address" " = 0x$test_value" $read_test
> +}

Patch is OK to me with these changes.

-- 
Yao (éå)