Merge pull request #510 from CippoX/warning-sourceaccurate

Removed warnings

Author: allevato

Sources/SwiftFormatCore/LegacyTriviaBehavior.swift

@@ -20,7 +20,7 @@ private final class LegacyTriviaBehaviorRewriter: SyntaxRewriter {
       token = token.with(\.leadingTrivia, pendingLeadingTrivia + token.leadingTrivia)
       self.pendingLeadingTrivia = nil
     }
-    if token.nextToken != nil,
+    if token.nextToken(viewMode: .sourceAccurate) != nil,
       let firstIndexToMove = token.trailingTrivia.firstIndex(where: shouldTriviaPieceBeMoved)
     {
       pendingLeadingTrivia = Trivia(pieces: Array(token.trailingTrivia[firstIndexToMove...]))

Sources/SwiftFormatCore/RuleMask.swift

@@ -136,7 +136,7 @@ fileprivate class RuleStatusCollectionVisitor: SyntaxVisitor {
   // MARK: - Syntax Visitation Methods
 
   override func visit(_ node: SourceFileSyntax) -> SyntaxVisitorContinueKind {
-    guard let firstToken = node.firstToken else {
+    guard let firstToken = node.firstToken(viewMode: .sourceAccurate) else {
       return .visitChildren
     }
     let comments = loneLineComments(in: firstToken.leadingTrivia, isFirstToken: true)
@@ -159,14 +159,14 @@ fileprivate class RuleStatusCollectionVisitor: SyntaxVisitor {
   }
 
   override func visit(_ node: CodeBlockItemSyntax) -> SyntaxVisitorContinueKind {
-    guard let firstToken = node.firstToken else {
+    guard let firstToken = node.firstToken(viewMode: .sourceAccurate) else {
       return .visitChildren
     }
     return appendRuleStatusDirectives(from: firstToken, of: Syntax(node))
   }
 
   override func visit(_ node: MemberDeclListItemSyntax) -> SyntaxVisitorContinueKind {
-    guard let firstToken = node.firstToken else {
+    guard let firstToken = node.firstToken(viewMode: .sourceAccurate) else {
       return .visitChildren
     }
     return appendRuleStatusDirectives(from: firstToken, of: Syntax(node))
@@ -183,7 +183,7 @@ fileprivate class RuleStatusCollectionVisitor: SyntaxVisitor {
   private func appendRuleStatusDirectives(from token: TokenSyntax, of node: Syntax)
     -> SyntaxVisitorContinueKind
   {
-    let isFirstInFile = token.previousToken == nil
+    let isFirstInFile = token.previousToken(viewMode: .sourceAccurate) == nil
     let matches = loneLineComments(in: token.leadingTrivia, isFirstToken: isFirstInFile)
       .compactMap(ruleStatusDirectiveMatch)
     let sourceRange = node.sourceRange(converter: sourceLocationConverter)

Sources/SwiftFormatPrettyPrint/TokenStreamCreator.swift

@@ -166,18 +166,18 @@ fileprivate final class AddModifierRewriter: SyntaxRewriter {
     for modifiersProvider: (NodeType) -> ModifierListSyntax?
   ) -> NodeType {
     guard let modifier = modifiersProvider(node)?.firstAndOnly,
-      let movingLeadingTrivia = modifier.nextToken?.leadingTrivia
+      let movingLeadingTrivia = modifier.nextToken(viewMode: .sourceAccurate)?.leadingTrivia
     else {
       // Otherwise, there's no trivia that needs to be relocated so the node is fine.
       return node
     }
     let nodeWithTrivia = replaceTrivia(
       on: node,
-      token: modifier.firstToken,
+      token: modifier.firstToken(viewMode: .sourceAccurate),
       leadingTrivia: movingLeadingTrivia)
     return replaceTrivia(
       on: nodeWithTrivia,
-      token: modifiersProvider(nodeWithTrivia)?.first?.nextToken,
+      token: modifiersProvider(nodeWithTrivia)?.first?.nextToken(viewMode: .sourceAccurate),
       leadingTrivia: [])
   }
 }

Sources/SwiftFormatRules/AddModifierRewriter.swift

@@ -15,7 +15,7 @@ import SwiftSyntax
 extension DeclSyntaxProtocol {
   /// Searches through the leading trivia of this decl for a documentation comment.
   var docComment: String? {
-    guard let tok = firstToken else { return nil }
+    guard let tok = firstToken(viewMode: .sourceAccurate) else { return nil }
     var comment = [String]()
 
     // We need to skip trivia until we see the first comment. This trivia will include all the

Sources/SwiftFormatRules/DeclSyntaxProtocol+Comments.swift

@@ -56,7 +56,7 @@ public final class DoNotUseSemicolons: SyntaxFormatRule {
       defer { newItems[idx] = newItem }
 
       // Check if the leading trivia for this statement needs a new line.
-      if previousHadSemicolon, let firstToken = newItem.firstToken,
+      if previousHadSemicolon, let firstToken = newItem.firstToken(viewMode: .sourceAccurate),
         !firstToken.leadingTrivia.containsNewlines
       {
         let leadingTrivia = .newlines(1) + firstToken.leadingTrivia

Sources/SwiftFormatRules/DoNotUseSemicolons.swift

@@ -53,14 +53,14 @@ public final class FullyIndirectEnum: SyntaxFormatRule {
     // If the `indirect` keyword being added would be the first token in the decl, we need to move
     // the leading trivia from the `enum` keyword to the new modifier to preserve the existing
     // line breaks/comments/indentation.
-    let firstTok = node.firstToken!
+    let firstTok = node.firstToken(viewMode: .sourceAccurate)!
     let leadingTrivia: Trivia
     let newEnumDecl: EnumDeclSyntax
 
     if firstTok.tokenKind == .keyword(.enum) {
       leadingTrivia = firstTok.leadingTrivia
       newEnumDecl = replaceTrivia(
-        on: node, token: node.firstToken, leadingTrivia: [])
+        on: node, token: node.firstToken(viewMode: .sourceAccurate), leadingTrivia: [])
     } else {
       leadingTrivia = []
       newEnumDecl = node
@@ -97,7 +97,7 @@ public final class FullyIndirectEnum: SyntaxFormatRule {
   ) -> EnumCaseDeclSyntax {
     if let modifiers = unformattedCase.modifiers, let first = modifiers.first {
       return replaceTrivia(
-        on: unformattedCase, token: first.firstToken, leadingTrivia: leadingTrivia
+        on: unformattedCase, token: first.firstToken(viewMode: .sourceAccurate), leadingTrivia: leadingTrivia
       )
     } else {
       return replaceTrivia(

Sources/SwiftFormatRules/FullyIndirectEnum.swift

@@ -72,12 +72,12 @@ extension ModifierListSyntax {
 
     if index == 0 {
       guard formatTrivia else { return inserting(modifier, at: index) }
-      guard let firstMod = first, let firstTok = firstMod.firstToken else {
+      guard let firstMod = first, let firstTok = firstMod.firstToken(viewMode: .sourceAccurate) else {
         return inserting(modifier, at: index)
       }
       let formattedMod = replaceTrivia(
         on: modifier,
-        token: modifier.firstToken,
+        token: modifier.firstToken(viewMode: .sourceAccurate),
         leadingTrivia: firstTok.leadingTrivia)
       newModifiers[0] = replaceTrivia(
         on: firstMod,

Sources/SwiftFormatRules/ModifierListSyntax+Convenience.swift

@@ -150,7 +150,7 @@ public final class NoCasesWithOnlyFallthrough: SyntaxFormatRule {
 
     // Check for any comments that are inline on the fallthrough statement. Inline comments are
     // always stored in the next token's leading trivia.
-    if let nextLeadingTrivia = onlyStatement.nextToken?.leadingTrivia,
+    if let nextLeadingTrivia = onlyStatement.nextToken(viewMode: .sourceAccurate)?.leadingTrivia,
       nextLeadingTrivia.prefix(while: { !$0.isNewline }).contains(where: { $0.isComment })
     {
       return false

Sources/SwiftFormatRules/NoCasesWithOnlyFallthrough.swift

@@ -29,7 +29,7 @@ public final class NoEmptyTrailingClosureParentheses: SyntaxFormatRule {
     {
       return super.visit(node)
     }
-    guard let name = node.calledExpression.lastToken?.with(\.leadingTrivia, []).with(\.trailingTrivia, []) else {
+    guard let name = node.calledExpression.lastToken(viewMode: .sourceAccurate)?.with(\.leadingTrivia, []).with(\.trailingTrivia, []) else {
       return super.visit(node)
     }
 
@@ -42,7 +42,7 @@ public final class NoEmptyTrailingClosureParentheses: SyntaxFormatRule {
     }
     let formattedExp = replaceTrivia(
       on: rewrittenCalledExpr,
-      token: rewrittenCalledExpr.lastToken,
+      token: rewrittenCalledExpr.lastToken(viewMode: .sourceAccurate),
       trailingTrivia: .spaces(1))
     let formattedClosure = visit(trailingClosure).as(ClosureExprSyntax.self)
     let result = node.with(\.leftParen, nil).with(\.rightParen, nil).with(\.calledExpression, formattedExp)