Forward port lint 1l and getNumber tweak

An additional message is emitted on `0_x`
because it asks to continue, but that is helpful.

Author: som-snytt

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

@@ -20,6 +20,8 @@ import config.Feature
 import config.Feature.migrateTo3
 import config.SourceVersion.`3.0`
 
+import java.lang.Character.isDigit
+
 object Scanners {
 
   /** Offset into source character array */
@@ -156,9 +158,9 @@ object Scanners {
       strVal = litBuf.toString
       litBuf.clear()
 
-    @inline def isNumberSeparator(c: Char): Boolean = c == '_'
+    inline def isNumberSeparator(c: Char): Boolean = c == '_'
 
-    @inline def removeNumberSeparators(s: String): String = if (s.indexOf('_') == -1) s else s.replace("_", "")
+    inline def removeNumberSeparators(s: String): String = if s.indexOf('_') == -1 then s else s.replace("_", "")
 
     // disallow trailing numeric separator char, but continue lexing
     def checkNoTrailingSeparator(): Unit =
@@ -849,7 +851,7 @@ object Scanners {
               //case 'b' | 'B' => base = 2  ; nextChar()
               case _         => base = 10 ; putChar('0')
             }
-            if (base != 10 && !isNumberSeparator(ch) && digit2int(ch, base) < 0)
+            if base != 10 && !isNumberSeparator(ch) && digit2int(ch, base) < 0 then
               error("invalid literal number")
           }
           fetchLeadingZero()
@@ -929,7 +931,7 @@ object Scanners {
         case '.' =>
           nextChar()
           if ('0' <= ch && ch <= '9') {
-            putChar('.'); getFraction(); setStrVal()
+            putChar('.'); getFraction()
           }
           else
             token = DOT
@@ -1389,7 +1391,7 @@ object Scanners {
     /** read fractional part and exponent of floating point number
      *  if one is present.
      */
-    protected def getFraction(): Unit = {
+    protected def getFraction(): Unit =
       token = DECILIT
       while ('0' <= ch && ch <= '9' || isNumberSeparator(ch)) {
         putChar(ch)
@@ -1427,41 +1429,44 @@ object Scanners {
         token = FLOATLIT
       }
       checkNoLetter()
-    }
+      setStrVal()
+    end getFraction
     def checkNoLetter(): Unit =
       if (isIdentifierPart(ch) && ch >= ' ')
         error("Invalid literal number")
 
     /** Read a number into strVal and set base
     */
-    protected def getNumber(): Unit = {
-      while (isNumberSeparator(ch) || digit2int(ch, base) >= 0) {
-        putChar(ch)
-        nextChar()
-      }
-      checkNoTrailingSeparator()
-      token = INTLIT
-      if (base == 10 && ch == '.') {
-        val lch = lookaheadChar()
-        if ('0' <= lch && lch <= '9') {
-          putChar('.')
-          nextChar()
-          getFraction()
-        }
-      }
-      else (ch: @switch) match {
-        case 'e' | 'E' | 'f' | 'F' | 'd' | 'D' =>
-          if (base == 10) getFraction()
-        case 'l' | 'L' =>
+    protected def getNumber(): Unit =
+      def consumeDigits(): Unit =
+        while isNumberSeparator(ch) || digit2int(ch, base) >= 0 do
+          putChar(ch)
           nextChar()
-          token = LONGLIT
-        case _ =>
-      }
+      // at dot with digit following
+      def restOfNonIntegralNumber(): Unit =
+        putChar('.')
+        nextChar()
+        getFraction()
+      // 1l is an acknowledged bad practice
+      def lintel(): Unit =
+        if ch == 'l' then
+          val msg = "Lowercase el for long is not recommended because it is easy to confuse with numeral 1; use uppercase L instead"
+          report.deprecationWarning(msg, sourcePos(offset + litBuf.length))
+      // after int: 5e7f, 42L, 42.toDouble but not 42b.
+      def restOfNumber(): Unit =
+        ch match
+        case 'e' | 'E' | 'f' | 'F' | 'd' | 'D' => getFraction()
+        case 'l' | 'L' => lintel() ; token = LONGLIT ; setStrVal() ; nextChar()
+        case _         => token = INTLIT  ; setStrVal() ; checkNoLetter()
+
+      // consume leading digits
+      consumeDigits()
 
       checkNoTrailingSeparator()
 
-      setStrVal()
-    }
+      val detectedFloat: Boolean = base == 10 && ch == '.' && isDigit(lookaheadChar())
+      if detectedFloat then restOfNonIntegralNumber() else restOfNumber()
+    end getNumber
 
     private def finishCharLit(): Unit = {
       nextChar()

tests/neg/t6124.check

@@ -43,9 +43,13 @@
    |             ^
    |             trailing separator is not allowed
 -- Error: tests/neg/t6124.scala:24:12 ----------------------------------------------------------------------------------
-24 |  val x5 = 0_x52 // error
+24 |  val x5 = 0_x52 // error // error
    |            ^
    |            trailing separator is not allowed
+-- Error: tests/neg/t6124.scala:24:11 ----------------------------------------------------------------------------------
+24 |  val x5 = 0_x52 // error // error
+   |           ^
+   |           Invalid literal number
 -- Error: tests/neg/t6124.scala:26:13 ----------------------------------------------------------------------------------
 26 |  val x8 = 0x52_ // error
    |             ^

tests/neg/t6124.scala

@@ -21,7 +21,7 @@ trait T {
   val x1 = _52 // error
   val x3 = 52_ // error
 
-  val x5 = 0_x52 // error
+  val x5 = 0_x52 // error // error
   val x6 = 0x_52
   val x8 = 0x52_ // error
   val x9 = 0_52