Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

inner class generation issue with RedBlack$$Empty #2873

Closed
scabug opened this issue Jan 8, 2010 · 8 comments
Closed

inner class generation issue with RedBlack$$Empty #2873

scabug opened this issue Jan 8, 2010 · 8 comments
Assignees
Milestone

Comments

@scabug
Copy link

@scabug scabug commented Jan 8, 2010

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
@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jan 8, 2010

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jan 10, 2011

@paulp said:
(In r23957) Test case closes #2873, no review.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented May 3, 2011

@hubertp said:
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.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 19, 2011

Commit Message Bot (anonymous) said:
(extempore in r25327) Restored test case for #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 #2873, no review.

@scabug scabug closed this Jul 19, 2011
@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 19, 2011

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

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 19, 2011

@paulp said:
Yes, but your rebuilding skills could use work.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 19, 2011

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

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 19, 2011

@paulp said:
Thank you however for asking me rather than pursuing any number of other avenues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.