On 01/20/2017 09:24 AM, Pedro Alves wrote:
> On 01/20/2017 02:37 PM, Luis Machado wrote:
>> Some leftover uppercase test names in py-xmethods.exp.  The patch also
>> replaces two "continue" calls with untested calls to make things a bit more
>> clear.
>>
>> gdb/testsuite/ChangeLog:
>>
>> 2017-01-20  Luis Machado  <lgustavo@codesourcery.com>
>>
>> 	* gdb.python/py-xmethods.exp: Fix test names starting with lowercase
>> 	and add untested calls.
>> ---
>>  gdb/testsuite/gdb.python/py-xmethods.exp | 44 +++++++++++++++++---------------
>>  1 file changed, 24 insertions(+), 20 deletions(-)
>>
>> diff --git a/gdb/testsuite/gdb.python/py-xmethods.exp b/gdb/testsuite/gdb.python/py-xmethods.exp
>> index 8f5bf16..38b5998 100644
>> --- a/gdb/testsuite/gdb.python/py-xmethods.exp
>> +++ b/gdb/testsuite/gdb.python/py-xmethods.exp
>> @@ -18,7 +18,9 @@
>>
>>  load_lib gdb-python.exp
>>
>> -if { [skip_cplus_tests] } { continue }
>> +if { [skip_cplus_tests] } {
>> +    untested "skipping C++ tests"
>> +}
>
> We still need something that makes the test
> actually be skipped.
>

Don't we? Oops.

How about the following?