From 399c17c78c3c2d010e0573bda4834af8c8957b76 Mon Sep 17 00:00:00 2001
From: Dave <snpukwana@gmail.com>
Date: Sat, 21 Sep 2019 00:23:58 -0400
Subject: [PATCH] Fix Swift 5.1 and Xcode 11 Warnings

---
 SwiftGraph.xcodeproj/project.pbxproj          |  8 ++++--
 .../xcschemes/SwiftGraph.xcscheme             | 28 ++++++++-----------
 .../SwiftGraphPerformanceTests.xcscheme       | 28 ++++++++-----------
 .../xcschemes/SwiftGraphSampleApp.xcscheme    | 24 +++++++---------
 .../SwiftGraphTests/DijkstraGraphTests.swift  |  6 ++--
 .../SwiftGraphCodableTests.swift              |  2 +-
 6 files changed, 43 insertions(+), 53 deletions(-)

diff --git a/SwiftGraph.xcodeproj/project.pbxproj b/SwiftGraph.xcodeproj/project.pbxproj
index 1bd4349..b5edc6a 100644
--- a/SwiftGraph.xcodeproj/project.pbxproj
+++ b/SwiftGraph.xcodeproj/project.pbxproj
@@ -436,7 +436,7 @@
 			attributes = {
 				LastSwiftMigration = 0710;
 				LastSwiftUpdateCheck = 0820;
-				LastUpgradeCheck = 1020;
+				LastUpgradeCheck = 1100;
 				ORGANIZATIONNAME = "Oak Snow Consulting";
 				TargetAttributes = {
 					55E620011A194C80000A5F7B = {
@@ -457,10 +457,9 @@
 			};
 			buildConfigurationList = 55E61FFD1A194C7F000A5F7B /* Build configuration list for PBXProject "SwiftGraph" */;
 			compatibilityVersion = "Xcode 3.2";
-			developmentRegion = English;
+			developmentRegion = en;
 			hasScannedForEncodings = 0;
 			knownRegions = (
-				English,
 				en,
 				Base,
 			);
@@ -722,6 +721,7 @@
 			buildSettings = {
 				ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES = YES;
 				ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
+				CODE_SIGN_IDENTITY = "-";
 				COMBINE_HIDPI_IMAGES = YES;
 				INFOPLIST_FILE = SwiftGraphSampleApp/Info.plist;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks";
@@ -738,6 +738,7 @@
 			buildSettings = {
 				ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES = YES;
 				ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
+				CODE_SIGN_IDENTITY = "-";
 				COMBINE_HIDPI_IMAGES = YES;
 				INFOPLIST_FILE = SwiftGraphSampleApp/Info.plist;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks";
@@ -986,6 +987,7 @@
 			buildSettings = {
 				ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES = YES;
 				ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
+				CODE_SIGN_IDENTITY = "-";
 				COMBINE_HIDPI_IMAGES = YES;
 				INFOPLIST_FILE = SwiftGraphSampleApp/Info.plist;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks";
diff --git a/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraph.xcscheme b/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraph.xcscheme
index eeae66e..31a873a 100644
--- a/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraph.xcscheme
+++ b/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraph.xcscheme
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Scheme
-   LastUpgradeVersion = "1020"
+   LastUpgradeVersion = "1100"
    version = "1.3">
    <BuildAction
       parallelizeBuildables = "YES"
@@ -26,8 +26,17 @@
       buildConfiguration = "Debug"
       selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
       selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
-      codeCoverageEnabled = "YES"
-      shouldUseLaunchSchemeArgsEnv = "YES">
+      shouldUseLaunchSchemeArgsEnv = "YES"
+      codeCoverageEnabled = "YES">
+      <MacroExpansion>
+         <BuildableReference
+            BuildableIdentifier = "primary"
+            BlueprintIdentifier = "7985B8FA1E5A4FB800C100E7"
+            BuildableName = "SwiftGraph.framework"
+            BlueprintName = "SwiftGraph"
+            ReferencedContainer = "container:SwiftGraph.xcodeproj">
+         </BuildableReference>
+      </MacroExpansion>
       <Testables>
          <TestableReference
             skipped = "NO">
@@ -40,17 +49,6 @@
             </BuildableReference>
          </TestableReference>
       </Testables>
-      <MacroExpansion>
-         <BuildableReference
-            BuildableIdentifier = "primary"
-            BlueprintIdentifier = "7985B8FA1E5A4FB800C100E7"
-            BuildableName = "SwiftGraph.framework"
-            BlueprintName = "SwiftGraph"
-            ReferencedContainer = "container:SwiftGraph.xcodeproj">
-         </BuildableReference>
-      </MacroExpansion>
-      <AdditionalOptions>
-      </AdditionalOptions>
    </TestAction>
    <LaunchAction
       buildConfiguration = "Debug"
@@ -71,8 +69,6 @@
             ReferencedContainer = "container:SwiftGraph.xcodeproj">
          </BuildableReference>
       </MacroExpansion>
-      <AdditionalOptions>
-      </AdditionalOptions>
    </LaunchAction>
    <ProfileAction
       buildConfiguration = "Release"
diff --git a/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphPerformanceTests.xcscheme b/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphPerformanceTests.xcscheme
index 4e434bc..4bc0ade 100644
--- a/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphPerformanceTests.xcscheme
+++ b/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphPerformanceTests.xcscheme
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Scheme
-   LastUpgradeVersion = "1020"
+   LastUpgradeVersion = "1100"
    version = "2.0">
    <BuildAction
       parallelizeBuildables = "YES"
@@ -40,9 +40,18 @@
       buildConfiguration = "Performance Tests"
       selectedDebuggerIdentifier = ""
       selectedLauncherIdentifier = "Xcode.IDEFoundation.Launcher.PosixSpawn"
+      shouldUseLaunchSchemeArgsEnv = "YES"
       disableMainThreadChecker = "YES"
-      systemAttachmentLifetime = "keepNever"
-      shouldUseLaunchSchemeArgsEnv = "YES">
+      systemAttachmentLifetime = "keepNever">
+      <MacroExpansion>
+         <BuildableReference
+            BuildableIdentifier = "primary"
+            BlueprintIdentifier = "7985B8FA1E5A4FB800C100E7"
+            BuildableName = "SwiftGraph.framework"
+            BlueprintName = "SwiftGraph"
+            ReferencedContainer = "container:SwiftGraph.xcodeproj">
+         </BuildableReference>
+      </MacroExpansion>
       <Testables>
          <TestableReference
             skipped = "NO">
@@ -55,17 +64,6 @@
             </BuildableReference>
          </TestableReference>
       </Testables>
-      <MacroExpansion>
-         <BuildableReference
-            BuildableIdentifier = "primary"
-            BlueprintIdentifier = "7985B8FA1E5A4FB800C100E7"
-            BuildableName = "SwiftGraph.framework"
-            BlueprintName = "SwiftGraph"
-            ReferencedContainer = "container:SwiftGraph.xcodeproj">
-         </BuildableReference>
-      </MacroExpansion>
-      <AdditionalOptions>
-      </AdditionalOptions>
    </TestAction>
    <LaunchAction
       buildConfiguration = "Performance Tests"
@@ -89,8 +87,6 @@
             ReferencedContainer = "container:SwiftGraph.xcodeproj">
          </BuildableReference>
       </MacroExpansion>
-      <AdditionalOptions>
-      </AdditionalOptions>
    </LaunchAction>
    <ProfileAction
       buildConfiguration = "Release"
diff --git a/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphSampleApp.xcscheme b/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphSampleApp.xcscheme
index 8b67164..495a441 100644
--- a/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphSampleApp.xcscheme
+++ b/SwiftGraph.xcodeproj/xcshareddata/xcschemes/SwiftGraphSampleApp.xcscheme
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Scheme
-   LastUpgradeVersion = "1020"
+   LastUpgradeVersion = "1100"
    version = "1.3">
    <BuildAction
       parallelizeBuildables = "YES"
@@ -27,6 +27,15 @@
       selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
       selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
       shouldUseLaunchSchemeArgsEnv = "YES">
+      <MacroExpansion>
+         <BuildableReference
+            BuildableIdentifier = "primary"
+            BlueprintIdentifier = "55E620011A194C80000A5F7B"
+            BuildableName = "SwiftGraphSampleApp.app"
+            BlueprintName = "SwiftGraphSampleApp"
+            ReferencedContainer = "container:SwiftGraph.xcodeproj">
+         </BuildableReference>
+      </MacroExpansion>
       <Testables>
          <TestableReference
             skipped = "NO">
@@ -39,17 +48,6 @@
             </BuildableReference>
          </TestableReference>
       </Testables>
-      <MacroExpansion>
-         <BuildableReference
-            BuildableIdentifier = "primary"
-            BlueprintIdentifier = "55E620011A194C80000A5F7B"
-            BuildableName = "SwiftGraphSampleApp.app"
-            BlueprintName = "SwiftGraphSampleApp"
-            ReferencedContainer = "container:SwiftGraph.xcodeproj">
-         </BuildableReference>
-      </MacroExpansion>
-      <AdditionalOptions>
-      </AdditionalOptions>
    </TestAction>
    <LaunchAction
       buildConfiguration = "Debug"
@@ -71,8 +69,6 @@
             ReferencedContainer = "container:SwiftGraph.xcodeproj">
          </BuildableReference>
       </BuildableProductRunnable>
-      <AdditionalOptions>
-      </AdditionalOptions>
    </LaunchAction>
    <ProfileAction
       buildConfiguration = "Release"
diff --git a/Tests/SwiftGraphTests/DijkstraGraphTests.swift b/Tests/SwiftGraphTests/DijkstraGraphTests.swift
index 58aa074..49f764e 100644
--- a/Tests/SwiftGraphTests/DijkstraGraphTests.swift
+++ b/Tests/SwiftGraphTests/DijkstraGraphTests.swift
@@ -95,7 +95,7 @@ class DijkstraGraphTests: XCTestCase {
         XCTAssertFalse(distances.isEmpty, "Dijkstra result set is empty.")
 
         //create map of distances to city names
-        var nameDistance: [String: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph)
+        let nameDistance: [String: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph)
         if let temp = nameDistance["San Francisco"] {
             XCTAssertEqual(temp!, 3057, "San Francisco should be 3057 miles away.")
         } else {
@@ -168,7 +168,7 @@ class DijkstraGraphTests: XCTestCase {
         XCTAssertFalse(distances.isEmpty, "Dijkstra result set is empty.")
         
         //create map of distances to city names
-        var nameDistance: [String: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph)
+        let nameDistance: [String: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph)
         if let temp = nameDistance["Seattle"] {
             XCTAssertEqual(temp!, 3455, "Seattle should be 3455 miles away.")
         } else {
@@ -207,7 +207,7 @@ class DijkstraGraphTests: XCTestCase {
         XCTAssertFalse(distances.isEmpty, "Dijkstra result set is empty.")
         
         //create map of distances to city names
-        var nameDistance: [String: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph2)
+        let nameDistance: [String: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph2)
         if let temp = nameDistance["Seattle"] {
             XCTAssertEqual(temp!, 2929, "Seattle should be 2929 miles away.")
         } else {
diff --git a/Tests/SwiftGraphTests/SwiftGraphCodableTests.swift b/Tests/SwiftGraphTests/SwiftGraphCodableTests.swift
index 1228c89..b61257f 100644
--- a/Tests/SwiftGraphTests/SwiftGraphCodableTests.swift
+++ b/Tests/SwiftGraphTests/SwiftGraphCodableTests.swift
@@ -137,7 +137,7 @@ extension SwiftGraphCodableTests {
         XCTAssertFalse(distances.isEmpty, "Dijkstra result set is empty.")
         
         //create map of distances to city names
-        var nameDistance: [SwiftGraphCodableTests_Vertex: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph)
+        let nameDistance: [SwiftGraphCodableTests_Vertex: Int?] = distanceArrayToVertexDict(distances: distances, graph: cityGraph)
         if let temp = nameDistance[vertexWithName("San Francisco")] {
             XCTAssertEqual(temp!, 3057, "San Francisco should be 3057 miles away.")
         } else {