Merge pull request #2750 from ahoppen/fix-warnings

Fix build warnings

Author: ahoppen

Sources/SwiftRefactor/SyntaxUtils.swift

@@ -11,7 +11,7 @@
 //===----------------------------------------------------------------------===//
 
 #if swift(>=6)
-public import SwiftSyntax
+internal import SwiftSyntax
 #else
 import SwiftSyntax
 #endif

Tests/PerformanceTest/ParsingPerformanceTests.swift

@@ -27,7 +27,7 @@ class ParsingPerformanceTests: XCTestCase {
   func testNativeParsingPerformance() throws {
     try XCTSkipIf(longTestsDisabled)
 
-    let source = try String(contentsOf: inputFile)
+    let source = try String(contentsOf: inputFile, encoding: .utf8)
 
     try measureInstructions {
       _ = Parser.parse(source: source)

Tests/PerformanceTest/SyntaxClassifierPerformanceTests.swift

@@ -28,7 +28,7 @@ class SyntaxClassifierPerformanceTests: XCTestCase {
   func testClassifierPerformance() throws {
     try XCTSkipIf(longTestsDisabled)
 
-    let source = try String(contentsOf: inputFile)
+    let source = try String(contentsOf: inputFile, encoding: .utf8)
     let parsed = Parser.parse(source: source)
 
     try measureInstructions {

Tests/PerformanceTest/VisitorPerformanceTests.swift

@@ -28,7 +28,7 @@ class VisitorPerformanceTests: XCTestCase {
     try XCTSkipIf(longTestsDisabled)
     class EmptyVisitor: SyntaxVisitor {}
 
-    let source = try String(contentsOf: inputFile)
+    let source = try String(contentsOf: inputFile, encoding: .utf8)
     let parsed = Parser.parse(source: source)
     let emptyVisitor = EmptyVisitor(viewMode: .sourceAccurate)
 
@@ -41,7 +41,7 @@ class VisitorPerformanceTests: XCTestCase {
     try XCTSkipIf(longTestsDisabled)
     class EmptyRewriter: SyntaxRewriter {}
 
-    let source = try String(contentsOf: inputFile)
+    let source = try String(contentsOf: inputFile, encoding: .utf8)
     let parsed = Parser.parse(source: source)
     let emptyRewriter = EmptyRewriter(viewMode: .sourceAccurate)
 
@@ -54,7 +54,7 @@ class VisitorPerformanceTests: XCTestCase {
     try XCTSkipIf(longTestsDisabled)
     class EmptyAnyVisitor: SyntaxAnyVisitor {}
 
-    let source = try String(contentsOf: inputFile)
+    let source = try String(contentsOf: inputFile, encoding: .utf8)
     let parsed = Parser.parse(source: source)
     let emptyVisitor = EmptyAnyVisitor(viewMode: .sourceAccurate)
 

Tests/SwiftParserTest/ParserTests.swift

@@ -61,20 +61,14 @@ class ParserTests: ParserTestCase {
     checkDiagnostics: Bool,
     shouldExclude: @Sendable (URL) -> Bool = { _ in false }
   ) {
-    // nonisolated(unsafe) because [URL] is not marked Sendable on Linux.
-    let _fileURLs = FileManager.default
+    let fileURLs = FileManager.default
       .enumerator(at: path, includingPropertiesForKeys: nil)!
       .compactMap({ $0 as? URL })
       .filter {
         $0.pathExtension == "swift"
           || $0.pathExtension == "sil"
           || $0.pathExtension == "swiftinterface"
       }
-    #if swift(>=6.0)
-    nonisolated(unsafe) let fileURLs = _fileURLs
-    #else
-    let fileURLs = _fileURLs
-    #endif
 
     print("\(name) - processing \(fileURLs.count) source files")
     DispatchQueue.concurrentPerform(iterations: fileURLs.count) { fileURLIndex in