Swift: Delete some TODO comments (that have been turned into issues).

Author: geoffw0

swift/ql/lib/codeql/swift/elements/decl/PrecedenceGroupDecl.qll

@@ -2,6 +2,6 @@ private import codeql.swift.generated.decl.PrecedenceGroupDecl
 
 class PrecedenceGroupDecl extends Generated::PrecedenceGroupDecl {
   override string toString() {
-    result = "precedencegroup ..." // TODO: Once we extract the name we can improve this.
+    result = "precedencegroup ..."
   }
 }

swift/ql/lib/codeql/swift/elements/expr/TupleElementExpr.qll

@@ -2,6 +2,6 @@ private import codeql.swift.generated.expr.TupleElementExpr
 
 class TupleElementExpr extends Generated::TupleElementExpr {
   override string toString() {
-    result = "." + this.getIndex() // TODO: Can be improved once we extract the name
+    result = "." + this.getIndex()
   }
 }

swift/ql/lib/codeql/swift/frameworks/StandardLibrary/FilePath.qll

@@ -15,7 +15,6 @@ class FilePath extends StructDecl {
  */
 private class FilePathSummaries extends SummaryModelCsv {
   override predicate row(string row) {
-    // TODO: Properly model this class
     row = ";FilePath;true;init(stringLiteral:);(String);;Argument[0];ReturnValue;taint"
   }
 }

swift/ql/lib/codeql/swift/frameworks/StandardLibrary/NsUrl.qll

@@ -10,7 +10,6 @@ private import codeql.swift.dataflow.ExternalFlow
  */
 private class NsUrlSummaries extends SummaryModelCsv {
   override predicate row(string row) {
-    // TODO: Properly model this class
     row = ";NSURL;true;init(string:);(String);;Argument[0];ReturnValue;taint"
   }
 }

swift/ql/lib/codeql/swift/frameworks/StandardLibrary/WebView.qll

@@ -138,7 +138,6 @@ private class JsValueSummaries extends SummaryModelCsv {
         ";JSValue;true;toRange();;;Argument[-1];ReturnValue;taint",
         ";JSValue;true;toRect();;;Argument[-1];ReturnValue;taint",
         ";JSValue;true;toSize();;;Argument[-1];ReturnValue;taint",
-        // TODO: These models could use content flow to be more precise
         ";JSValue;true;atIndex(_:);;;Argument[-1];ReturnValue;taint",
         ";JSValue;true;defineProperty(_:descriptor:);;;Argument[1];Argument[-1];taint",
         ";JSValue;true;forProperty(_:);;;Argument[-1];ReturnValue;taint",

swift/ql/lib/codeql/swift/security/CleartextStoragePreferencesExtensions.qll

@@ -63,10 +63,12 @@ private class NSUbiquitousKeyValueStore extends CleartextStoragePreferencesSink
  * A more complicated case, this is a macOS-only way of writing to
  * NSUserDefaults by modifying the `NSUserDefaultsController.values: Any`
  * object via reflection (`perform(Selector)`) or the `NSKeyValueCoding`,
- * `NSKeyValueBindingCreation` APIs. (TODO)
+ * `NSKeyValueBindingCreation` APIs.
  */
 private class NSUserDefaultsControllerStore extends CleartextStoragePreferencesSink {
-  NSUserDefaultsControllerStore() { none() }
+  NSUserDefaultsControllerStore() {
+    none() // not yet implemented
+  }
 
   override string getStoreName() { result = "the user defaults database" }
 }

swift/ql/lib/codeql/swift/security/PathInjectionExtensions.qll

@@ -39,7 +39,6 @@ private class DefaultPathInjectionSink extends PathInjectionSink {
 private class DefaultPathInjectionSanitizer extends PathInjectionSanitizer {
   DefaultPathInjectionSanitizer() {
     // This is a simplified implementation.
-    // TODO: Implement a complete path sanitizer when Guards are available.
     exists(CallExpr starts, CallExpr normalize, DataFlow::Node validated |
       starts.getStaticTarget().getName() = "starts(with:)" and
       starts.getStaticTarget().getEnclosingDecl() instanceof FilePath and