From aebb5b4a4cf541281378fdf3a6f2455579ec538b Mon Sep 17 00:00:00 2001
From: Basile Burg <basile.b@gmx.com>
Date: Sun, 16 Sep 2018 16:58:39 +0200
Subject: [PATCH] fix #696 - Unused variable fake positive with UFCS

Apparently it was a bug not only specific to the UFCS.
Now every use of the member access operator is handled.
---
 src/dscanner/analysis/unused.d | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/dscanner/analysis/unused.d b/src/dscanner/analysis/unused.d
index 9f211ab..42ddcb9 100644
--- a/src/dscanner/analysis/unused.d
+++ b/src/dscanner/analysis/unused.d
@@ -199,11 +199,10 @@ final class UnusedVariableCheck : BaseAnalyzer
 
 	override void visit(const UnaryExpression unary)
 	{
-		if (unary.prefix == tok!"*")
-			interestDepth++;
+		const bool interesting = unary.prefix == tok!"*" || unary.unaryExpression !is null;
+		interestDepth += interesting;
 		unary.accept(this);
-		if (unary.prefix == tok!"*")
-			interestDepth--;
+		interestDepth -= interesting;
 	}
 
 	override void visit(const MixinExpression mix)
@@ -539,6 +538,12 @@ private:
 		mixin("decl++;");
 	}
 
+	void main()
+	{
+	    const int testValue;
+	    testValue.writeln;
+	}
+
 	}c, sac);
 	stderr.writeln("Unittest for UnusedVariableCheck passed.");
 }