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

c.typeCheck doesn't see declarations from the current scope #14

Closed
xeno-by opened this issue Dec 31, 2013 · 10 comments
Closed

c.typeCheck doesn't see declarations from the current scope #14

xeno-by opened this issue Dec 31, 2013 · 10 comments

Comments

@xeno-by
Copy link
Member Author

xeno-by commented Dec 31, 2013

There's an interesting twist to this story. If one tries to typecheck something that refers to the type parameters of the annottee, these type parameters wouldn't be taken into account: aztek/scala-workflow#2 (comment)

xeno-by added a commit that referenced this issue Jan 2, 2014
xeno-by added a commit that referenced this issue Jan 2, 2014
xeno-by added a commit that referenced this issue Jan 2, 2014
xeno-by added a commit that referenced this issue Jan 2, 2014
@FlorianKirmaier
Copy link

since 2.0.0-M8 the situation is worse. The compiler crashes with the following error:
scala.reflect.internal.Symbols$CyclicReference: illegal cyclic reference involving object TNodeASDF

Here is a example: http://pastebin.com/raw.php?i=Svm11P10

@aparo
Copy link

aparo commented Jan 30, 2014

@FlorianKirmaier you are using java 1.8.0-ea. How is it stable?
Kudos to you of using 2.11-M8 on Java 1.8.0-ea :-)

@xeno-by
Copy link
Member Author

xeno-by commented Jan 31, 2014

Thank you! I'll try to take a look today.

@xeno-by
Copy link
Member Author

xeno-by commented Feb 1, 2014

@aparo Yeah, I can reproduce the error. I'll see what I can do.

@xeno-by
Copy link
Member Author

xeno-by commented Feb 1, 2014

@FlorianKirmaier I get the same error in the most recent paradise for M7. When did it stop working for you?

@xeno-by
Copy link
Member Author

xeno-by commented Feb 1, 2014

I see now. This problem is a direct consequence of the fix I applied above. It only manifests itself when there's an import in scope, which is why I missed it in the first place.

@xeno-by
Copy link
Member Author

xeno-by commented Feb 1, 2014

@FlorianKirmaier @aparo Fixed the regression, started jenkins builds. The fix should be published with an updated 2.0.0-SNAPSHOT build for all supported Scala versions (2.10.3, 2.10.4-SNAPSHOT, 2.11.0-M7, 2.11.0-M8, 2.11.0-SNAPSHOT) within a couple hours.

@xeno-by xeno-by added this to the 2.1.0-final milestone Mar 22, 2014
@xeno-by
Copy link
Member Author

xeno-by commented Mar 25, 2014

Unfortunately I haven't made any progress here, so we won't have a fix in 2.0.0-final. Any ideas are very welcome!

@xeno-by
Copy link
Member Author

xeno-by commented Nov 19, 2015

I've merged this issue into #75 and will close this issue now. If this is a blocker for you, please let me know, and we'll think of something.

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

No branches or pull requests

3 participants