Initial changes with references to MaxClassFileLength and variants replaced with MaxNameLength and variants hardcoded to 240

Author: IDoCodingStuffs

compiler/src/dotty/tools/dotc/config/ScalaSettings.scala

@@ -64,7 +64,6 @@ class ScalaSettings extends Settings.SettingGroup {
   val Xhelp: Setting[Boolean] = BooleanSetting("-X", "Print a synopsis of advanced options.")
   val XnoForwarders: Setting[Boolean] = BooleanSetting("-Xno-forwarders", "Do not generate static forwarders in mirror classes.")
   val XmaxInlines: Setting[Int] = IntSetting("-Xmax-inlines", "Maximal number of successive inlines", 32)
-  val XmaxClassfileName: Setting[Int] = IntSetting("-Xmax-classfile-name", "Maximum filename length for generated classes", 255, 72 to 255)
   val Xmigration: Setting[ScalaVersion] = VersionSetting("-Xmigration", "Warn about constructs whose behavior may have changed since version.")
   val Xprint: Setting[List[String]] = PhasesSetting("-Xprint", "Print out program after")
   val XprintTypes: Setting[Boolean] = BooleanSetting("-Xprint-types", "Print tree types (debugging option).")

compiler/src/dotty/tools/dotc/core/NameOps.scala

@@ -27,7 +27,7 @@ object NameOps {
      */
     def apply(s: String)(implicit ctx: Context): String = {
       val marker = "$$$$"
-      val limit: Int = ctx.settings.XmaxClassfileName.value
+      final val LIMIT: Int = 240
       val MaxNameLength = (limit - 6) min 2 * (limit - 6 - 2 * marker.length - 32)
 
       def toMD5(s: String, edge: Int): String = {

sbt-dotty/sbt-test/source-dependencies/compactify/build.sbt

@@ -2,9 +2,4 @@ TaskKey[Unit]("output-empty") := {
   val outputDirectory = (classDirectory in Compile).value
   val classes = (outputDirectory ** "*.class").get
   if (classes.nonEmpty) sys.error("Classes existed:\n\t" + classes.mkString("\n\t")) else ()
-}
-
-// apparently Travis CI stopped allowing long file names
-// it fails with the default setting of 255 characters so
-// we have to set lower limit ourselves
-scalacOptions ++= Seq("-Xmax-classfile-name", "240")
+}

tests/pending/run/t8199.scala

@@ -39,8 +39,10 @@ object Test extends dotty.runtime.LegacyApp {
     checkClassName(c.getName)
   }
   def checkClassName(name: String): Unit = {
-    val defaultMaxClassFileLength = 255
-    assert((name + ".class").length <= defaultMaxClassFileLength, name)
+    final val MaxSuffixLength = 7 // "$.class".length + 1 // potential module class suffix and file extension
+    final val MaxNameLength   = 240 - MaxSuffixLength
+
+    assert((name + ".class").length <= defaultMaxNameLength, name)
   }
   def checkCallerImplClassName(): Unit = {
     val name = Thread.currentThread.getStackTrace.apply(2).getClassName