[Bug objc/12516] New: [3.4/Regression][ObjC] Prototype visability

[Bug objc/12516] New: [3.4/Regression][ObjC] Prototype visability

[d dot ayers at inode dot at]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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

           Summary: [3.4/Regression][ObjC] Prototype visability
           Product: gcc
           Version: 3.4
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: objc
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: d dot ayers at inode dot at
                CC: gcc-bugs at gcc dot gnu dot org
  GCC host triplet: *-*-*

Private @implementation methods of the main @implementation become fully visable
to any following code.  The compiler should warn if they are used outside of the
class.

[Bug objc/12516] [3.4/Regression][ObjC] Prototype visability

[d dot ayers at inode dot at]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From d dot ayers at inode dot at  2003-10-05 14:10 -------
Created an attachment (id=4890)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=4890&action=view)
Testcase

[Bug objc/12516] [3.4 Regression] Prototype visability

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic
            Summary|[3.4/Regression][ObjC]      |[3.4 Regression] Prototype
                   |Prototype visability        |visability


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-10-05 16:13 -------
It was caused by this patch:
2003-10-03  Alexander Malmberg  <alexander@malmberg.org>
            Ziemowit Laski  <zlaski@apple.com>

        * objc/objc-act.c (add_method_to_hash_list, lookup_category):
        New functions.
        (lookup_method_in_hash_lists): New parameter indicating whether
        we are messaging 'Class' or 'id'.
        (check_duplicates): Likewise; do not assume all methods will
        be either class or instance methods.
        (generate_category, finish_class): Use lookup_category().
        (add_method): Use add_method_to_hash_list(); insert instance
        methods of root classes into the global class method hash table.
        (add_category): Use lookup_category(); avoid constructing
        duplicate categories.
        (really_start_method): Add method to corresponding @interface,
        if not already there (and if the @interface exists).
        (finish_message_expr, finish_objc): Adjust calls to
        check_duplicates().


I want to say this is correct behavior though (but I do not want to get into a fight over this), I will let 
someone else decide this one.

[Bug objc/12516] [3.4 Regression] Prototype visability

[d dot ayers at inode dot at]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


d dot ayers at inode dot at changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |INVALID


------- Additional Comments From d dot ayers at inode dot at  2003-10-08 09:19 -------
The patch removes the warning which was previously emitted when a method
undeclared on an @interface section is used outside of it's @implementation
context.  The warning is inappropriately worded for this case.

A patch for a more meaningful warning is under discussion.