-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Fix SI-4581. #1201
Fix SI-4581. #1201
Conversation
Specifically, the final flag on the generated static field is no longer ommitted. Fix 2 failing test-cases.
Started jenkins job pr-rangepos at https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/289/ |
Started jenkins job pr-scala-testsuite-linux-opt at https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/998/ |
jenkins job pr-rangepos: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/289/ |
jenkins job pr-scala-testsuite-linux-opt: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/998/ |
Ping @paulp |
@@ -768,7 +773,8 @@ abstract class CleanUp extends Transform with ast.TreeDSL { | |||
staticSym <- clazz.info.decls | |||
if staticSym.hasStaticAnnotation | |||
} staticSym match { | |||
case stfieldSym if stfieldSym.isVariable => | |||
case stfieldSym if (stfieldSym.isValue && !stfieldSym.isMethod) || stfieldSym.isVariable => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there are (very recently) methods such that this logic could be if x.isVal || x.isVar.
Looks OK to me. |
Specifically, the final flag on the generated static field
is no longer ommitted.
Fix 2 failing test-cases.
Empty check file
t4581.check
is there until we fix https://issues.scala-lang.org/browse/SI-6289.Review by @paulp.