[XcodeGen] Make 'Byte' comform 'ExpressibleByUnicodeScalarLiteral'

Instead of comparing with 'UnicodeScalar', construct 'Byte' from
literals and compare with them. Make it less evil :)

Author: rintaro

utils/swift-xcodegen/Sources/SwiftXcodeGen/Ninja/NinjaBuildFile.swift

@@ -154,7 +154,7 @@ extension NinjaBuildFile {
 
 extension Byte {
   fileprivate var isNinjaVarName: Bool {
-    switch self.scalar {
+    switch self {
     case "0"..."9", "a"..."z", "A"..."Z", "_", "-":
       return true
     default:

utils/swift-xcodegen/Sources/SwiftXcodeGen/Scanner/Byte.swift

@@ -17,26 +17,16 @@ struct Byte: Hashable {
   }
 }
 
-// Please forgive me...
-func == (lhs: UnicodeScalar, rhs: Byte?) -> Bool {
-  guard let rhs else { return false }
-  return lhs.value == rhs.rawValue
-}
-func == (lhs: Byte?, rhs: UnicodeScalar) -> Bool {
-  rhs == lhs
-}
-func != (lhs: UnicodeScalar, rhs: Byte?) -> Bool {
-  !(lhs == rhs)
-}
-func != (lhs: Byte?, rhs: UnicodeScalar) -> Bool {
-  rhs != lhs
+extension Byte: ExpressibleByUnicodeScalarLiteral {
+  init(unicodeScalarLiteral value: UnicodeScalar) {
+    self.init(UInt8(ascii: value))
+  }
 }
 
-func ~= (pattern: UnicodeScalar, match: Byte) -> Bool {
-  pattern == match
-}
-func ~= (pattern: UnicodeScalar, match: Byte?) -> Bool {
-  pattern == match
+extension Byte: Comparable {
+  static func < (lhs: Self, rhs: Self) -> Bool {
+    lhs.rawValue < rhs.rawValue
+  }
 }
 
 extension Byte? {
@@ -61,14 +51,4 @@ extension Byte {
   var isSpaceTabOrNewline: Bool {
     isSpaceOrTab || isNewline
   }
-  init(ascii scalar: UnicodeScalar) {
-    assert(scalar.isASCII)
-    self.rawValue = UInt8(scalar.value)
-  }
-  var scalar: UnicodeScalar {
-    UnicodeScalar(UInt32(rawValue))!
-  }
-  var char: Character {
-    .init(scalar)
-  }
 }

utils/swift-xcodegen/Sources/SwiftXcodeGen/Scanner/ByteScanner.swift

@@ -77,10 +77,6 @@ struct ByteScanner {
     tryEat(where: { $0 == byte })
   }
 
-  mutating func tryEat(_ c: UnicodeScalar) -> Bool {
-    tryEat(where: { $0 == c })
-  }
-
   mutating func tryEat<S: Sequence>(_ seq: S) -> Bool where S.Element == UInt8 {
     let start = cursor
     for byte in seq {

utils/swift-xcodegen/Sources/SwiftXcodeGen/Utils.swift

@@ -78,7 +78,7 @@ extension String {
       let result = scanner.consumeWhole { consumer in
         switch consumer.peek {
         case "\\", "\"":
-          consumer.append(Byte(ascii: "\\"))
+          consumer.append("\\")
         case " ", "$": // $ is potentially a variable reference
           needsQuotes = true
         default: