Scala Programming Language
  1. Scala Programming Language
  2. SI-2873

inner class generation issue with RedBlack$$Empty

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Scala 2.10.0
    • Component/s: Misc Compiler
    • Labels:
      None

      Description

      I'm not sure yet why Redblack$$Empty is the only class I've seen this with (it's not the only case object nested in an abstract class by any means) but no inner class entry is generated for it, with the following among the consequences:

      scala> classOf[scala.collection.immutable.RedBlack[_]].getMethod("Empty").getGenericReturnType
      java.lang.TypeNotPresentException: Type scala.collection.immutable.RedBlack$$Empty not present
      	at sun.reflect.generics.factory.CoreReflectionFactory.makeNamedType(CoreReflectionFactory.java:98)
      	at sun.reflect.generics.visitor.Reifier.visitClassTypeSignature(Reifier.java:129)
      	at sun.reflect.generics.tree.ClassTypeSignature.accept(ClassTypeSignature.java:31)
      	at sun.reflect.generics.repository.MethodRepository.getReturnType(MethodRepository.java:50)
      

      Here are the inner classes:

      public abstract SI-113= SI-91 of SI-11; //Tree=class scala/collection/immutable/RedBlack$$Tree of class scala/collection/immutable/RedBlack
      public SI-27= SI-59 of SI-11; //RedTree=class scala/collection/immutable/RedBlack$$RedTree of class scala/collection/immutable/RedBlack
      public abstract SI-116= SI-115 of SI-11; //NonEmpty=class scala/collection/immutable/RedBlack$$NonEmpty of class scala/collection/immutable/RedBlack
      public SI-36= SI-54 of SI-11; //BlackTree=class scala/collection/immutable/RedBlack$$BlackTree of class scala/collection/immutable/RedBlack
      

        Activity

        Hide
        Paul Phillips added a comment -

        (In r23957) Test case closes SI-2873, no review.

        Show
        Paul Phillips added a comment - (In r23957) Test case closes SI-2873 , no review.
        Hide
        Hubert Plociniczak added a comment -

        Reopening because of r24868. This vanished because of some changes done in nested objects -> lazy als translation but since we are back to the original idea, this bug magically re-appears.

        Show
        Hubert Plociniczak added a comment - Reopening because of r24868. This vanished because of some changes done in nested objects -> lazy als translation but since we are back to the original idea, this bug magically re-appears.
        Hide
        Commit Message Bot added a comment -

        (extempore in r25327) Restored test case for SI-2873 which hubert had callously deleted
        back when it was briefly irrelevant. It would have been failing were
        it around, but now it passes. Closes SI-2873, no review.

        Show
        Commit Message Bot added a comment - (extempore in r25327 ) Restored test case for SI-2873 which hubert had callously deleted back when it was briefly irrelevant. It would have been failing were it around, but now it passes. Closes SI-2873 , no review.
        Hide
        Simon Ochsenreither added a comment -

        This still fails with java.lang.TypeNotPresentException in 2.10.0.r25323-b20110719020429. Is this really fixed?

        Show
        Simon Ochsenreither added a comment - This still fails with java.lang.TypeNotPresentException in 2.10.0.r25323-b20110719020429. Is this really fixed?
        Hide
        Paul Phillips added a comment -

        Yes, but your rebuilding skills could use work.

        Show
        Paul Phillips added a comment - Yes, but your rebuilding skills could use work.
        Hide
        Paul Phillips added a comment -

        Oh, and it's fixed in r25326, i.e. the commit immediately preceding it.

        Show
        Paul Phillips added a comment - Oh, and it's fixed in r25326, i.e. the commit immediately preceding it.
        Hide
        Paul Phillips added a comment -

        Thank you however for asking me rather than pursuing any number of other avenues.

        Show
        Paul Phillips added a comment - Thank you however for asking me rather than pursuing any number of other avenues.

          People

          • Assignee:
            Iulian Dragos
            Reporter:
            Paul Phillips
            TracCC:
            Ismael Juma, Johannes Rudolph, Paul Phillips
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development