[Bug libobjc/49882] New: class_getSuperClass() returns nil on a newly allocated, but not registered, class

[Bug libobjc/49882] New: class_getSuperClass() returns nil on a newly allocated, but not registered, class

[nicola at gcc dot gnu.org]

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

           Summary: class_getSuperClass() returns nil on a newly
                    allocated, but not registered, class
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libobjc
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: nicola@gcc.gnu.org


>From Ludovic Marcotte <ludovic@Sophos.ca> --

> There's a bug with the gcc 4.6 runtime since objc_getSuperClass() on a newly 
> allocated - but not registered - class returns nil.

Unfortunately, this is actually a documented behaviour of the GCC 4.6 runtime,
so 
we can't really change it in GCC 4.6.x.

But if this is how the Apple runtime behaves, in 4.7.0 we should change our 
runtime (and the corresponding documentation) to match.

Thanks

[Bug libobjc/49882] class_getSuperClass() returns nil on a newly allocated, but not registered, class

[nicola at gcc dot gnu.org]

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

--- Comment #1 from Nicola Pero <nicola at gcc dot gnu.org> 2011-08-06 09:49:33 UTC ---
Author: nicola
Date: Sat Aug  6 09:49:30 2011
New Revision: 177505

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=177505
Log:
In libobjc/:
2011-08-06  Nicola Pero  <nicola.pero@meta-innovation.com>

    PR libobjc/49882
    * class.c (class_getSuperclass): Return the superclass if the
    class is in construction.
    * objc/runtime.h (class_getSuperclass): Updated documentation.

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

    PR libobjc/49882
    * objc.dg/gnu-api-2-class.m (main): Test class_getSuperclass()
    with classes that are in construction.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/objc.dg/gnu-api-2-class.m
    trunk/libobjc/ChangeLog
    trunk/libobjc/class.c
    trunk/libobjc/objc/runtime.h

[Bug libobjc/49882] class_getSuperClass() returns nil on a newly allocated, but not registered, class

[nicola at gcc dot gnu.org]

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

--- Comment #2 from Nicola Pero <nicola at gcc dot gnu.org> 2011-08-06 14:20:13 UTC ---
Author: nicola
Date: Sat Aug  6 14:20:09 2011
New Revision: 177510

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=177510
Log:
In libobjc/:
2011-08-06  Nicola Pero  <nicola.pero@meta-innovation.com>

    PR libobjc/50002
    * class.c (__objc_update_classes_with_methods): Iterate over meta
    classes as well as normal classes when refreshing the method
    implementations.  This fixes replacing class methods.

2011-08-06  Nicola Pero  <nicola.pero@meta-innovation.com>

    * class.c (class_getSuperclass): Fixed to work with meta classes
    still in construction too.

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

    PR libobjc/50002
    * objc.dg/gnu-api-2-class.m: Updated comments.
    * obj-c++.dg/gnu-api-2-class.mm: Likewise.
    * objc.dg/gnu-api-2-class-meta.m: New test.
    * obj-c++.dg/gnu-api-2-class-meta.mm: Likewise.

2011-08-06  Nicola Pero  <nicola.pero@meta-innovation.com>

    PR libobjc/49882
    * obj-c++.dg/gnu-api-2-class.mm (main): Test class_getSuperclass()
    with classes that are in construction.

Added:
    trunk/gcc/testsuite/obj-c++.dg/gnu-api-2-class-meta.mm
    trunk/gcc/testsuite/objc.dg/gnu-api-2-class-meta.m
Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/obj-c++.dg/gnu-api-2-class.mm
    trunk/gcc/testsuite/objc.dg/gnu-api-2-class.m
    trunk/libobjc/ChangeLog
    trunk/libobjc/class.c

[Bug libobjc/49882] class_getSuperClass() returns nil on a newly allocated, but not registered, class

[nicola at gcc dot gnu.org]

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

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

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

--- Comment #3 from Nicola Pero <nicola at gcc dot gnu.org> 2011-08-06 14:21:29 UTC ---
Confirmed, then fixed in trunk (which will be 4.7.x).

Thanks