Refactor code and check for errors in typeIdent

Author: nicolasstucki

compiler/src/dotty/tools/dotc/parsing/Parsers.scala

@@ -543,24 +543,22 @@ object Parsers {
       }
 
     /** Accept identifier and return Ident with its name as a term name. */
-    def termIdent(): Ident = {
-      val lastOffset = in.lastOffset
-      val id = atPos(in.offset) {
-        makeIdent(in.token, ident())
-      }
-      // Make sure that even trees with parsing errors have a offset that is within the offset
-      if (id.name == nme.ERROR && id.pos == NoPosition) atPos(lastOffset - 1)(id)
-      else id
+    def termIdent(): Ident = atPos(in.offset) {
+      makeIdent(in.token, ident())
     }
 
     /** Accept identifier and return Ident with its name as a type name. */
     def typeIdent(): Ident = atPos(in.offset) {
       makeIdent(in.token, ident().toTypeName)
     }
 
-    private def makeIdent(tok: Token, name: Name) =
-      if (tok == BACKQUOTED_IDENT) BackquotedIdent(name)
-      else Ident(name)
+    private def makeIdent(tok: Token, name: Name) = {
+      val tree =
+        if (tok == BACKQUOTED_IDENT) BackquotedIdent(name)
+        else Ident(name)
+      if (name.toTermName == nme.ERROR) tree.withPos(Position(in.offset, in.offset))
+      else tree
+    }
 
     def wildcardIdent(): Ident =
       atPos(accept(USCORE)) { Ident(nme.WILDCARD) }