Rewrite: TypeMatcher warning about hack

Author: MasseGuillaume

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

@@ -27,6 +27,8 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
     } yield (open, close)
 
   // 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 {
@@ -37,8 +39,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)))
     }
   }