Imported From: https://issues.scala-lang.org/browse/SI-4788?orig=1
Reporter: Arjun Panday (joune)
Affected Versions: 2.8.1
See #5420

@paulp said:
This should be easy to fix, assigning to meeting looking for a victim.

@soc said:
This is related to (and could probably be the cause of) #5420.

@adriaanm said:
Un-assigning to foster work stealing, as announced in https://groups.google.com/forum/?fromgroups=#!topic/scala-internals/o8WG4plpNkw

@adriaanm said:
Unassigning and rescheduling to M6 as previous deadline was missed.

@soc said:
Funny coincidence ... I just fixed this and #5420 5 minutes ago. Is it ok if I create a PR on Thursday?

@adriaanm said:
Sure, doesn't seem like it cause regressions. Thanks!

@soc said:
scala/scala#3459

@soc said (edited on Oct 1, 2014 4:35:29 PM UTC):
Just tried to update this patch to finally get rid of this ticket: https://github.com/soc/scala/tree/SI-4788-new ... apart from the known issue of scalac thinking that classOf doesn't exist in Predef, the test suite now fails with over 400 failures + nondeterminism:

test/partest --failed

Selected 402 tests drawn from previously failed tests

# starting 60 tests in pos
ok  1 - pos/NoCyclicReference.scala             
ok  2 - pos/A.scala                             
ok  3 - pos/S1.scala                            
ok  4 - pos/FPTest.scala                        
ok  5 - pos/S3.scala                            
ok  6 - pos/List1.scala                         
ok  7 - pos/MailBox.scala                       
!!  8 - pos/inline-access-levels                  [compilation failed]
...

Many of the failures are related to annotated elements just "disappearing":

@foo val bar = 1
//Somewhere else:
println(bar)
// scalac: Hey dude, bar doesn't exist, amiright?!

But not all of the failures use annotations.

I'm running out of ideas how a fix to make scalac determine correctly whether a Java annotation should be visible at runtime can wreak such a havoc all over the place.

I'd be happy about any help in this regard.

@lrytz said:
scala/scala#4026

@lrytz said:
Need to re-open this.

In scala/scala#5606 we upgraded partest to a version that changes the mixed compilation mode. This caused a change in the retained annotations, which (probably) should not be the case.

I'll take a look.

@lrytz said:
The fix in scala/scala#4026 only works under separate compilation, because annotations are skipped by the Java parser. We have an open issue for this: #8928, so this one can be closed.

@Retention(value=SOURCE) // not parsed by the Java parser
@interface SAnnotation {}

Tests updated accordingly in scala/scala#5606.