Upgrade libdparse and DCD

.github/workflows/default.yml

@@ -70,13 +70,6 @@ jobs:
         include:
           - { do_report: 1, build: { type: dub, version: 'current' }, host: 'ubuntu-22.04', compiler: { version: dmd-latest, dmd: dmd } }
 
-          - compiler:
-              dmd: dmd
-            host: macos-13
-            build:
-              type: 'dub'
-              version: 'current'
-
     runs-on: ${{ matrix.host }}
 
     steps:

dub.json

@@ -11,8 +11,8 @@
     "built_with_dub"
   ],
   "dependencies": {
-    "libdparse": ">=0.23.1 <0.26.0",
-    "dcd:dsymbol": ">=0.16.0-beta.2 <0.17.0",
+    "libdparse": ">=0.24.0 <0.26.0",
+    "dcd:dsymbol": ">=0.16.2 <0.17.0",
     "inifiled": "~>1.3.1",
     "emsi_containers": "~>0.9.0",
     "libddoc": "~>0.8.0"

dub.selections.json

@@ -1,12 +1,12 @@
 {
 	"fileVersion": 1,
 	"versions": {
-		"dcd": "0.16.0-beta.2",
+		"dcd": "0.16.2",
 		"dsymbol": "0.13.0",
 		"emsi_containers": "0.9.0",
 		"inifiled": "1.3.3",
 		"libddoc": "0.8.0",
-		"libdparse": "0.25.0",
+		"libdparse": "0.25.1",
 		"stdx-allocator": "2.77.5"
 	}
 }

src/dscanner/analysis/mismatched_args.d

@@ -1,12 +1,12 @@
 module dscanner.analysis.mismatched_args;
 
+import dparse.ast;
+import dparse.lexer : tok, Token;
 import dscanner.analysis.base;
 import dscanner.utils : safeAccess;
+import dsymbol.builtin.names;
 import dsymbol.scope_;
 import dsymbol.symbol;
-import dparse.ast;
-import dparse.lexer : tok, Token;
-import dsymbol.builtin.names;
 
 /// Checks for mismatched argument and parameter names
 final class MismatchedArgumentCheck : BaseAnalyzer
@@ -22,8 +22,6 @@ final class MismatchedArgumentCheck : BaseAnalyzer
 	override void visit(const FunctionCallExpression fce)
 	{
 		import std.typecons : scoped;
-		import std.algorithm.iteration : each, map;
-		import std.array : array;
 
 		if (fce.arguments is null)
 			return;
@@ -53,7 +51,7 @@ final class MismatchedArgumentCheck : BaseAnalyzer
 		{
 			// The cast is a hack because .array() confuses the compiler's overload
 			// resolution code.
-			const(istring)[] params = sym is null ? [] : sym.argNames[].map!(a => cast() a).array();
+			const(istring)[] params = sym is null ? [] : sym.argNames;
 			const ArgMismatch[] mismatches = compareArgsToParams(params, args);
 			if (mismatches.length == 0)
 				matched = true;
@@ -251,8 +249,8 @@ unittest
 
 unittest
 {
+	import dscanner.analysis.config : Check, disabledConfig, StaticAnalysisConfig;
 	import dscanner.analysis.helpers : assertAnalyzerWarnings;
-	import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
 	import std.stdio : stderr;
 
 	StaticAnalysisConfig sac = disabledConfig();

src/dscanner/analysis/unused.d

@@ -7,10 +7,10 @@ module dscanner.analysis.unused;
 import dparse.ast;
 import dparse.lexer;
 import dscanner.analysis.base;
-import std.container;
-import std.regex : Regex, regex, matchAll;
 import dsymbol.scope_ : Scope;
 import std.algorithm : all;
+import std.container;
+import std.regex : matchAll, regex, Regex;
 
 /**
  * Checks for unused variables.
@@ -86,10 +86,10 @@ abstract class UnusedIdentifierCheck : BaseAnalyzer
 
 	override void visit(const SwitchStatement switchStatement)
 	{
-		if (switchStatement.expression !is null)
+		if (switchStatement.condition !is null && switchStatement.condition.expression !is null)
 		{
 			interestDepth++;
-			switchStatement.expression.accept(this);
+			switchStatement.condition.expression.accept(this);
 			interestDepth--;
 		}
 		switchStatement.accept(this);