Merge pull request #282 from ThoughtWorksInc/fix-280

Don't transform macros (fix #280)

compilerplugins-ResetEverywhere/build.sbt

@@ -7,3 +7,5 @@ scalacOptions ++= {
     Seq()
   }
 }
+
+scalacOptions in Test += raw"""-Xplugin:${(packageBin in Compile).value}"""

compilerplugins-ResetEverywhere/src/main/scala/com/thoughtworks/dsl/compilerplugins/ResetEverywhere.scala

@@ -100,6 +100,8 @@ final class ResetEverywhere(override val global: Global) extends Plugin {
           body.mapConserve {
             case valDef: ValDef if !valDef.mods.isParamAccessor =>
               transformRootValDef(valDef)
+            case defDef: DefDef if defDef.mods.hasFlag(Flag.MACRO) =>
+              defDef
             case initializer: TermTree =>
               annotatedReset(initializer)
             case stat =>

compilerplugins-ResetEverywhere/src/test/scala/com/thoughtworks/dsl/compilerplugins/Issue280.scala

@@ -0,0 +1,20 @@
+package com.thoughtworks.dsl.compilerplugins
+
+object Issue280 {
+  import scala.language.experimental.macros
+
+  object ContainsMacro {
+
+    def apply: String = macro Macros.apply
+
+  }
+
+  class Macros(val c: scala.reflect.macros.blackbox.Context) {
+
+    import c.universe._
+
+    def apply: Tree = q"???"
+
+  }
+
+}

keywords-Yield/src/test/scala/com/thoughtworks/dsl/keywords/YieldSpec.scala

@@ -13,6 +13,17 @@ import scala.runtime.NonLocalReturnControl
   */
 class YieldSpec extends FreeSpec with Matchers {
 
+  {
+    !Yield(1)
+
+    def nested(): Stream[Int] = {
+      !Yield(2)
+      Stream.empty[Int]
+    }
+
+    nested()
+  } // Should compile
+
   "Given a continuation that uses Yield and Each expressions" - {
 
     def asyncFunction: Stream[String] !! Unit = _ {