Merge pull request #9666 from dotty-staging/fix-4588

Enable Windows CI

Author: liufengyun

.appveyor.yml

@@ -15,4 +15,6 @@ build_script:
   - cmd: sbt dotty-bootstrapped/compile
 test_script:
   - cmd: sbt "dotty-bootstrapped/dotc tests\pos\HelloWorld.scala"
+  # - cmd: sbt test
+  # - cmd: sbt dotty-bootstrapped/test
   - cmd: sbt sjsJUnitTests/test

compiler/src/dotty/tools/dotc/Run.scala

@@ -12,25 +12,25 @@ import Denotations.Denotation
 import typer.Typer
 import typer.ImportInfo._
 import Decorators._
-import io.{AbstractFile, PlainFile}
+import io.{AbstractFile, PlainFile, VirtualFile}
 import Phases.unfusedPhases
 
-import scala.io.Codec
 import util._
 import reporting.Reporter
 import rewrites.Rewrites
-import java.io.{BufferedWriter, OutputStreamWriter}
 
 import profile.Profiler
 import printing.XprintMode
 import parsing.Parsers.Parser
 import parsing.JavaParsers.JavaParser
 import typer.ImplicitRunInfo
-import collection.mutable
 
-import dotty.tools.io.VirtualFile
+import java.io.{BufferedWriter, OutputStreamWriter}
+import java.nio.charset.StandardCharsets
 
+import scala.collection.mutable
 import scala.util.control.NonFatal
+import scala.io.Codec
 
 /** A compiler run. Exports various methods to compile source files */
 class Run(comp: Compiler, ictx: Context) extends ImplicitRunInfo with ConstraintRunInfo {
@@ -274,7 +274,7 @@ class Run(comp: Compiler, ictx: Context) extends ImplicitRunInfo with Constraint
       val uuid = java.util.UUID.randomUUID().toString
       val ext = if (isJava) ".java" else ".scala"
       val virtualFile = new VirtualFile(s"compileFromString-$uuid.$ext")
-      val writer = new BufferedWriter(new OutputStreamWriter(virtualFile.output, "UTF-8")) // buffering is still advised by javadoc
+      val writer = new BufferedWriter(new OutputStreamWriter(virtualFile.output, StandardCharsets.UTF_8.name)) // buffering is still advised by javadoc
       writer.write(source)
       writer.close()
       new SourceFile(virtualFile, Codec.UTF8)

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

@@ -6,6 +6,7 @@ import scala.annotation.internal.sharable
 
 import java.io.IOException
 import java.util.jar.Attributes.{ Name => AttributeName }
+import java.nio.charset.StandardCharsets
 
 /** Loads `library.properties` from the jar. */
 object Properties extends PropertiesTrait {
@@ -88,13 +89,13 @@ trait PropertiesTrait {
   /** This is the encoding to use reading in source files, overridden with -encoding
    *  Note that it uses "prop" i.e. looks in the scala jar, not the system properties.
    */
-  def sourceEncoding: String        = scalaPropOrElse("file.encoding", "UTF-8")
+  def sourceEncoding: String        = scalaPropOrElse("file.encoding", StandardCharsets.UTF_8.name)
   def sourceReader: String          = scalaPropOrElse("source.reader", "scala.tools.nsc.io.SourceReader")
 
   /** This is the default text encoding, overridden (unreliably) with
    *  `JAVA_OPTS="-Dfile.encoding=Foo"`
    */
-  def encodingString: String        = propOrElse("file.encoding", "UTF-8")
+  def encodingString: String        = propOrElse("file.encoding", StandardCharsets.UTF_8.name)
 
   /** The default end of line character.
    */

compiler/src/dotty/tools/dotc/core/tasty/CommentPickler.scala

@@ -7,7 +7,7 @@ import dotty.tools.dotc.core.Contexts._
 import dotty.tools.tasty.TastyBuffer
 import TastyBuffer.{Addr, NoAddr}
 
-import java.nio.charset.Charset
+import java.nio.charset.StandardCharsets
 
 class CommentPickler(pickler: TastyPickler, addrOfTree: tpd.Tree => Addr, docString: untpd.MemberDef => Option[Comment]):
   private val buf = new TastyBuffer(5000)
@@ -17,7 +17,7 @@ class CommentPickler(pickler: TastyPickler, addrOfTree: tpd.Tree => Addr, docStr
 
   private def pickleComment(addr: Addr, comment: Comment): Unit =
     if addr != NoAddr then
-      val bytes = comment.raw.getBytes(Charset.forName("UTF-8"))
+      val bytes = comment.raw.getBytes(StandardCharsets.UTF_8)
       val length = bytes.length
       buf.writeAddr(addr)
       buf.writeNat(length)

compiler/src/dotty/tools/dotc/core/tasty/CommentUnpickler.scala

@@ -8,7 +8,7 @@ import util.HashMap
 import dotty.tools.tasty.{TastyReader, TastyBuffer}
 import TastyBuffer.Addr
 
-import java.nio.charset.Charset
+import java.nio.charset.StandardCharsets
 
 class CommentUnpickler(reader: TastyReader) {
   import reader._
@@ -21,7 +21,7 @@ class CommentUnpickler(reader: TastyReader) {
       if (length > 0) {
         val bytes = readBytes(length)
         val position = new Span(readLongInt())
-        val rawComment = new String(bytes, Charset.forName("UTF-8"))
+        val rawComment = new String(bytes, StandardCharsets.UTF_8)
         comments(addr) = Comment(position, rawComment)
       }
     }

compiler/src/dotty/tools/dotc/decompiler/DecompilationPrinter.scala

@@ -2,6 +2,7 @@ package dotty.tools.dotc
 package decompiler
 
 import java.io.{OutputStream, PrintStream}
+import java.nio.charset.StandardCharsets
 
 import scala.io.Codec
 
@@ -27,7 +28,7 @@ class DecompilationPrinter extends Phase {
       var ps: PrintStream = null
       try {
         os = File(outputDir.fileNamed("decompiled.scala").path)(Codec.UTF8).outputStream(append = true)
-        ps = new PrintStream(os, /* autoFlush = */ false, "UTF-8")
+        ps = new PrintStream(os, /* autoFlush = */ false, StandardCharsets.UTF_8.name)
         printToOutput(ps)
       }
       finally {

compiler/src/dotty/tools/dotc/interactive/InteractiveDriver.scala

@@ -6,6 +6,7 @@ import java.net.URI
 import java.io._
 import java.nio.file._
 import java.nio.file.attribute.BasicFileAttributes
+import java.nio.charset.StandardCharsets
 import java.util.zip._
 
 import scala.collection._
@@ -298,7 +299,7 @@ class InteractiveDriver(val settings: List[String]) extends Driver {
   private def toSource(uri: URI, sourceCode: String): SourceFile = {
     val path = Paths.get(uri)
     val virtualFile = new VirtualFile(path.getFileName.toString, path.toString)
-    val writer = new BufferedWriter(new OutputStreamWriter(virtualFile.output, "UTF-8"))
+    val writer = new BufferedWriter(new OutputStreamWriter(virtualFile.output, StandardCharsets.UTF_8.name))
     writer.write(sourceCode)
     writer.close()
     new SourceFile(virtualFile, Codec.UTF8)

compiler/src/dotty/tools/dotc/semanticdb/internal/SemanticdbInputStream.scala

@@ -3,6 +3,8 @@ package dotty.tools.dotc.semanticdb.internal
 import java.io.IOException
 import java.io.InputStream
 import java.util.Arrays
+import java.nio.charset.StandardCharsets
+
 import SemanticdbInputStream._
 
 import scala.collection.mutable
@@ -434,15 +436,15 @@ class SemanticdbInputStream private (buffer: Array[Byte], input: InputStream) {
   def readString(): String = {
     val size: Int = readRawVarint32()
     if (size <= (bufferSize - bufferPos) && size > 0) {
-      val result: String = new String(buffer, bufferPos, size, Internal.UTF_8)
+      val result: String = new String(buffer, bufferPos, size, StandardCharsets.UTF_8)
       bufferPos += size
       return result
     }
     else if (size == 0) {
       return ""
     }
     else {
-      return new String(readRawBytesSlowPath(size), Internal.UTF_8)
+      return new String(readRawBytesSlowPath(size), StandardCharsets.UTF_8)
     }
   }
 

compiler/src/dotty/tools/dotc/semanticdb/internal/SemanticdbOutputStream.scala

@@ -3,6 +3,8 @@ package dotty.tools.dotc.semanticdb.internal
 import java.io.IOException
 import java.io.OutputStream
 import java.nio.ByteBuffer
+import java.nio.charset.StandardCharsets
+
 import SemanticdbOutputStream._
 
 object SemanticdbOutputStream {
@@ -144,7 +146,7 @@ object SemanticdbOutputStream {
   def computeBoolSizeNoTag(value: Boolean): Int = 1
 
   def computeStringSizeNoTag(value: String): Int = {
-    val bytes = value.getBytes(Internal.UTF_8)
+    val bytes = value.getBytes(StandardCharsets.UTF_8)
     computeRawVarint32Size(bytes.length) + bytes.length
   }
 
@@ -391,7 +393,7 @@ class SemanticdbOutputStream(output: OutputStream, buffer: Array[Byte]) {
   }
 
   def writeStringNoTag(value: String): Unit = {
-    val bytes = value.getBytes(Internal.UTF_8)
+    val bytes = value.getBytes(StandardCharsets.UTF_8)
     writeRawVarint32(bytes.length)
     writeRawBytes(bytes)
   }
@@ -602,6 +604,5 @@ class SemanticdbOutputStream(output: OutputStream, buffer: Array[Byte]) {
 }
 
 object Internal {
-  val UTF_8 = java.nio.charset.Charset.forName("UTF-8")
   val EMPTY_BYTE_ARRAY: Array[Byte] = Array()
 }

compiler/src/dotty/tools/dotc/util/ShowPickled.scala

@@ -2,9 +2,11 @@ package dotty.tools.dotc
 package util
 
 import java.io.PrintStream
+import java.nio.charset.StandardCharsets
 import java.lang.Long.toHexString
 import java.lang.Float.intBitsToFloat
 import java.lang.Double.longBitsToDouble
+
 import core.unpickleScala2.PickleBuffer
 import core.Names._
 
@@ -18,7 +20,7 @@ object ShowPickled {
       case _                                                                            => false
     }
     def readName: String =
-      if (isName) new String(bytes, "UTF-8")
+      if (isName) new String(bytes, StandardCharsets.UTF_8)
       else sys.error("%s is no name" format tagName)
     def nameIndex: Int =
       if (hasName) readNat(bytes, 0)