[Bug objc/47314] New: Incorrect multiple selectors warning

[Bug objc/47314] New: Incorrect multiple selectors warning

[nicola at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47314

           Summary: Incorrect multiple selectors warning
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: objc
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: nicola@gcc.gnu.org


A trivial typo in objc-act.c means that the following testcase emits the wrong
warning when compiled with -Wselector:

/* Contributed by Nicola Pero <nicola.pero@meta-innovation.com>, January 2011. 
*/
/* { dg-options "-Wselector" } */
/* { dg-do compile } */

#include <objc/objc.h>

@interface MyObject
- (void) method;
@end

@interface MyObject2;
- (int) method;
@end

GCC 4.6.0 produces the warning

selector-warn-1.m:13:1: warning: multiple selectors named ‘+method’ found
[enabled by default]
selector-warn-1.m:8:1: note: found ‘-(void)method’
selector-warn-1.m:12:1: note: also found ‘-(int)method’

Note that the selector is wrongly referred to as '+method' instead of
'-method'.

Thanks

[Bug objc/47314] Incorrect multiple selectors warning

[nicola at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47314

Nicola Pero <nicola at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.01.16 13:24:03
         AssignedTo|unassigned at gcc dot       |nicola at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from Nicola Pero <nicola at gcc dot gnu.org> 2011-01-16 13:24:03 UTC ---
Mine; I have a patch already.

Thanks

[Bug objc/47314] Incorrect multiple selectors warning

[nicola at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47314

--- Comment #2 from Nicola Pero <nicola at gcc dot gnu.org> 2011-01-17 22:17:50 UTC ---
Author: nicola
Date: Mon Jan 17 22:17:47 2011
New Revision: 168934

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168934
Log:
In gcc/objc/:
2011-01-17  Nicola Pero  <nicola.pero@meta-innovation.com>

    PR objc/47314
    * objc-act.c (finish_objc): When calling check_duplicates to check
    duplicated instance methods, set 'is_class' to 0, not 1.

In gcc/testsuite/:
2011-01-17  Nicola Pero  <nicola.pero@meta-innovation.com>

    PR objc/47314
    * objc.dg/selector-warn-1.m: New.
    * obj-c++.dg/selector-warn-1.mm: New.

Added:
    trunk/gcc/testsuite/obj-c++.dg/selector-warn-1.mm
    trunk/gcc/testsuite/objc.dg/selector-warn-1.m
Modified:
    trunk/gcc/objc/ChangeLog
    trunk/gcc/objc/objc-act.c
    trunk/gcc/testsuite/ChangeLog

[Bug objc/47314] Incorrect multiple selectors warning

[nicola at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47314

Nicola Pero <nicola at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Known to work|                            |4.6.0
         Resolution|                            |FIXED
   Target Milestone|---                         |4.6.0

--- Comment #3 from Nicola Pero <nicola at gcc dot gnu.org> 2011-01-19 02:54:56 UTC ---
Fixed in trunk (4.6.0).

Thanks