Files
SwiftLint/Source/SwiftLintFramework/Rules/OperatorFunctionWhitespaceRule.swift
T
JP Simard 989127cbe0 [StyleViolation] use RuleDescription's description when reason is nil
Many cases just used a static string that was nearly identical to the rule
description as the `reason` parameter when initializing a StyleViolation.
2015-11-17 10:25:57 -08:00

49 lines
2.0 KiB
Swift

//
// OperatorWhitespaceRule.swift
// SwiftLint
//
// Created by Akira Hirakawa on 8/6/15.
// Copyright (c) 2015 Realm. All rights reserved.
//
import SourceKittenFramework
public struct OperatorFunctionWhitespaceRule: Rule {
public init() {}
public static let description = RuleDescription(
identifier: "operator_whitespace",
name: "Operator Function Whitespace",
description: "Operators should be surrounded by a single whitespace when defining them.",
nonTriggeringExamples: [
"func <| (lhs: Int, rhs: Int) -> Int {}\n",
"func <|< <A>(lhs: A, rhs: A) -> A {}\n",
"func abc(lhs: Int, rhs: Int) -> Int {}\n"
],
triggeringExamples: [
"func <|(lhs: Int, rhs: Int) -> Int {}\n", // no spaces after
"func <|<<A>(lhs: A, rhs: A) -> A {}\n", // no spaces after
"func <| (lhs: Int, rhs: Int) -> Int {}\n", // 2 spaces after
"func <|< <A>(lhs: A, rhs: A) -> A {}\n", // 2 spaces after
"func <| (lhs: Int, rhs: Int) -> Int {}\n", // 2 spaces before
"func <|< <A>(lhs: A, rhs: A) -> A {}\n" // 2 spaces before
]
)
public func validateFile(file: File) -> [StyleViolation] {
let operators = ["/", "=", "-", "+", "!", "*", "|", "^", "~", "?", "."].map({"\\\($0)"}) +
["%", "<", ">", "&"]
let zeroOrManySpaces = "(\\s{0}|\\s{2,})"
let pattern1 = "func\\s+[" + operators.joinWithSeparator("") +
"]+\(zeroOrManySpaces)(<[A-Z]+>)?\\("
let pattern2 = "func\(zeroOrManySpaces)[" + operators.joinWithSeparator("") +
"]+\\s+(<[A-Z]+>)?\\("
return file.matchPattern("(\(pattern1)|\(pattern2))").filter { _, syntaxKinds in
return syntaxKinds.first == .Keyword
}.map { range, _ in
return StyleViolation(ruleDescription: self.dynamicType.description,
location: Location(file: file, offset: range.location))
}
}
}