Re: gdb.objc/objcdecode.exp test error..

[Matt Rice <ratmice at gmail dot com>]

On Wed, Sep 23, 2009 at 5:53 PM, Joel Brobecker <brobecker@adacore.com> wrote:
<snip>

> ? ?(gdb) break create
>
> can result in numerous breakpoints being set if create just happens
> to be the name any ObjC method. And as it turns out, the NSThread
> class has a method called "main"...
>
> I really think that allowing the above shortcut is a mis-feature
> that we should consider removing. We could possibly think about
> introducing another syntax meaning all "create" methods in all
> classes, if it really is needed by ObjC developers.

here is a patch which does this....

the previous patches i'd sent (category.diff) i keep finding failures
that aren't covered by my tests.

so i'll keep working on that and getting more complete test coverage,
I will have to update the tests that I had previously sent

because they tested the various failure modes of this feature mostly,
if we remove it, there are lots of invalid tests in there that need to
be tweaked.

2009-09-24  Matt Rice  <ratmice@gmail.com>

     * linespec.c: (is_objc_method_format): Allow '[' .... ']' formatted
     objc methods.
    (decode_line_1): Don't call decode_objc unless its an objc
    method format.

Attachment: disable-misfeature.patch
Description: Binary data