Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

[nomaster] doesn't touch NonemptyAttachments

NonemptyAttachments in macro API used to be Attachments$NonemptyAttachments.
However it's private, so noone outside scala.reflect.macros can access it,
making it ineligible for the incompatibility criterion.
  • Loading branch information...
commit 7bf0ecc8d2d92b352182d61095bf564253f813d3 1 parent 3d318be
@xeno-by xeno-by authored
Showing with 2 additions and 0 deletions.
  1. +2 −0  src/reflect/scala/reflect/macros/Attachments.scala
View
2  src/reflect/scala/reflect/macros/Attachments.scala
@@ -56,6 +56,8 @@ abstract class Attachments { self =>
// SI-7018: This used to be an inner class of `Attachments`, but that led to a memory leak in the
// IDE via $outer pointers.
+// Forward compatibility note: This class used to be Attachments$NonemptyAttachments.
+// However it's private, therefore it transcends the compatibility policy for 2.10.x.
private final class NonemptyAttachments[P >: Null](override val pos: P, override val all: Set[Any]) extends Attachments {
type Pos = P
def withPos(newPos: Pos) = new NonemptyAttachments(newPos, all)

1 comment on commit 7bf0ecc

@scala-jenkins

Job pr-checkin-per-commit failed for 7bf0ecc (results):


Took 9 s.
sad kitty
to rebuild, comment "PLS REBUILD/pr-checkin-per-commit@7bf0ecc"on PR #2112

Please sign in to comment.
Something went wrong with that request. Please try again.