Avoid overcompilation involving inline or annotation trees

We used to pretty-print trees in the API info we send to sbt, but the
pretty-printed output seems to be unstable leading to overcompilation,
so just use the raw trees instead (this should also be faster).

Author: smarter

compiler/src/dotty/tools/dotc/sbt/ExtractAPI.scala

@@ -595,12 +595,11 @@ private class ExtractAPICollector(implicit val ctx: Context) extends ThunkHolder
     if (!inlineBody.isEmpty) {
       // FIXME: If the body of an inlineable method changes, all the reverse
       // dependencies of this method need to be recompiled. sbt has no way
-      // of tracking method bodies, so as a hack we include the pretty-printed
-      // typed tree of the method as part of the signature we send to sbt.
+      // of tracking method bodies, so as a hack we include the printed
+      // tree of the method as part of the signature we send to sbt.
       // To do this properly we would need a way to hash trees and types in
       // dotty itself.
-      val printTypesCtx = ctx.fresh.setSetting(ctx.settings.XprintTypes, true)
-      annots += marker(inlineBody.show(printTypesCtx))
+      annots += marker(inlineBody.toString)
     }
 
     // In the Scala2 ExtractAPI phase we only extract annotations that extend
@@ -624,6 +623,6 @@ private class ExtractAPICollector(implicit val ctx: Context) extends ThunkHolder
     // junit tests are annotated @org.junit.Test).
     api.Annotation.of(
       apiType(annot.tree.tpe), // Used by sbt to find tests to run
-      Array(api.AnnotationArgument.of("FULLTREE", annot.tree.show)))
+      Array(api.AnnotationArgument.of("FULLTREE", annot.tree.toString)))
   }
 }