Merge pull request scala#4700 from dotty-staging/decompile-fix-setters

Do not print setters
astojanov · Jun 22, 2018 · 89010f2 · 89010f2
2 parents 4725fe5 + e63bbb2
commit 89010f2
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/library/src/scala/tasty/util/ShowSourceCode.scala b/library/src/scala/tasty/util/ShowSourceCode.scala
@@ -141,12 +141,12 @@ class ShowSourceCode[T <: Tasty with Singleton](tasty0: T) extends Show[T](tasty
             })
           }
           def isInnerModuleObject = d.flags.isLazy && d.flags.isObject
-          !flags.isParam && !flags.isParamAccessor && !isCaseClassUnOverridableMethod && !isInnerModuleObject
+          !flags.isParam && !flags.isParamAccessor && !flags.isFieldAccessor && !isCaseClassUnOverridableMethod && !isInnerModuleObject
         }
         val stats1 = stats.collect {
-          case stat@Definition() if keepDefinition(stat) => stat
-          case stat@Import(_, _) => stat
-          case stat@Term() => stat
+          case stat @ Definition() if keepDefinition(stat) => stat
+          case stat @ Import(_, _) => stat
+          case stat @ Term() => stat
         }
 
         def printBody(printSelf: Boolean) = {

diff --git a/tests/pos/simpleSetter.decompiled b/tests/pos/simpleSetter.decompiled
@@ -0,0 +1,5 @@
+/** Decompiled from out/posTestFromTasty/pos/simpleSetter/Test.class */
+object Test {
+  var name: scala.Predef.String = "foo"
+  Test.name = Test.name.+("123")
+}
diff --git a/tests/pos/simpleSetter.scala b/tests/pos/simpleSetter.scala
@@ -0,0 +1,4 @@
+object Test {
+  var name: String = "foo"
+  name = name + "123"
+}