Remove commented out code

egregius313 · egregius313 · commit ceb169bc8b13 · 2024-02-21T12:51:13.000-05:00
diff --git a/csharp/ql/lib/semmle/code/csharp/security/dataflow/flowsources/Local.qll b/csharp/ql/lib/semmle/code/csharp/security/dataflow/flowsources/Local.qll
@@ -44,14 +44,6 @@ abstract class CommandLineArgumentSource extends LocalFlowSource {
   override string getSourceType() { result = "command line argument" }
 }
 
-// private class SystemEnvironmentCommandLineSource extends CommandLineArgumentSource {
-//   SystemEnvironmentCommandLineSource() {
-//     exists(PropertyAccess pa | this.asExpr() = pa |
-//       pa.getTarget().hasName("CommandLine") and
-//       pa.getTarget().getDeclaringType().hasFullyQualifiedName("System", "Environment")
-//     )
-//   }
-// }
 private class MainMethodArgumentSource extends CommandLineArgumentSource {
   MainMethodArgumentSource() { this.asParameter() = any(MainMethod mainMethod).getAParameter() }
 }

Original file line number	Diff line number	Diff line change
`@@ -44,14 +44,6 @@ abstract class CommandLineArgumentSource extends LocalFlowSource {`
`44`	`44`	`override string getSourceType() { result = "command line argument" }`
`45`	`45`	`}`
`46`	`46`
`47`		`-// private class SystemEnvironmentCommandLineSource extends CommandLineArgumentSource {`
`48`		`-// SystemEnvironmentCommandLineSource() {`
`49`		`-// exists(PropertyAccess pa \| this.asExpr() = pa \|`
`50`		`-// pa.getTarget().hasName("CommandLine") and`
`51`		`-// pa.getTarget().getDeclaringType().hasFullyQualifiedName("System", "Environment")`
`52`		`-// )`
`53`		`-// }`
`54`		`-// }`
`55`	`47`	`private class MainMethodArgumentSource extends CommandLineArgumentSource {`
`56`	`48`	`MainMethodArgumentSource() { this.asParameter() = any(MainMethod mainMethod).getAParameter() }`
`57`	`49`	`}`