Rewrite: TypeMatcher warning about hack

Author: MasseGuillaume

scalafix/rules/src/main/scala/fix/Scalacollectioncompat_newcollections.scala

@@ -8,7 +8,8 @@ import scala.meta._
 case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
   extends SemanticRule(index, "Scalacollectioncompat_newcollections") {
 
-  // terms dont give us terms https://github.com/scalameta/scalameta/issues/1212
+  // WARNING: TOTAL HACK
+  // this is only to unblock us until Term.tpe is available: https://github.com/scalameta/scalameta/issues/1212
   // if we have a simple identifier, we can look at his definition at query it's type
   // this should be improved in future version of scalameta
   object TypeMatcher {
@@ -19,8 +20,7 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
   final class TypeMatcher(symbols: Symbol*)(implicit index: SemanticdbIndex) {
     def unapply(tree: Tree): Boolean = {
       index.denotation(tree)
-           .map(_.names.headOption.exists(n => symbols.exists(_ == n.symbol)))
-           .getOrElse(false)
+           .exists(_.names.headOption.exists(n => symbols.exists(_ == n.symbol)))
     }
   }