Re: [PATCH] fix PR-15501

[Pedro Alves <palves@redhat.com>]

On 08/22/2013 02:13 PM, Waqas, Muhammad wrote:

> +# Perform tests for disable/enable commands on multiple
> +# locations and breakpoints.
> +#
> +# WHAT     - the command to test (disable/enable).

That extra whitespace after WHAT was there before to align
with the other arguments.  There are no other arguments now,
so write:

# WHAT - the command to test (disable/enable).

> +    if { "$what" == "enable" } {
> +	# Set variable values for enable command.
> +	set oppos "disable"
> +	set oppos_res "n"
> +	set what_res "y"
> +	set p1 "fail"
> +	set p2 "pass"
> +

Spurious extra line.

> +    }
> +
> +    # Now enable(disable) $b.1 $b2.1.
> +    gdb_test_no_output "$what $b1.1 $b2.1" "$what \$b1.1 \$b2.1"
> +    set test1 "${what}d \$b1.1 and \$b2.1"
> +
> +    # Now $b1.1 and $b2.1 should be enabled(disabled)

Missing period.  (Most other sentences had it.)

> +
> +    # Now $b4 $b3.1 should be enabled(disabled) and
> +    # $b3 should remain disabled(enabled)

Ditto.

OK with those changes.  Thanks!

As the final patch will have changes, please commit, and
still repost it.

-- 
Pedro Alves