Merge pull request #225 from Hackerpilot/simd

Simd

libdparse

@@ -1 +1 @@
-Subproject commit bcf115f811bb5b2a99d09eb3b8df139102d2d982
+Subproject commit b64fb8c91efa17a895567403969d4fc87adaa42a

src/analysis/local_imports.d

@@ -19,6 +19,9 @@ class LocalImportCheck : BaseAnalyzer
 {
 	alias visit = BaseAnalyzer.visit;
 
+	/**
+	 * Construct with the given file name.
+	 */
 	this(string fileName)
 	{
 		super(fileName);

src/analysis/numbers.d

@@ -17,8 +17,12 @@ import analysis.helpers;
  */
 class NumberStyleCheck : BaseAnalyzer
 {
+public:
 	alias visit = BaseAnalyzer.visit;
 
+	/**
+	 * Constructs the style checker with the given file name.
+	 */
 	this(string fileName)
 	{
 		super(fileName);
@@ -26,7 +30,7 @@ class NumberStyleCheck : BaseAnalyzer
 
 	override void visit(const Token t)
 	{
-		import std.algorithm;
+		import std.algorithm : startsWith;
 		if (isNumberLiteral(t.type) && !t.text.startsWith("0x")
 			&& ((t.text.startsWith("0b") && !t.text.matchFirst(badBinaryRegex).empty)
 				|| !t.text.matchFirst(badDecimalRegex).empty))
@@ -35,14 +39,14 @@ class NumberStyleCheck : BaseAnalyzer
 				"Use underscores to improve number constant readability.");
 		}
 	}
-
+private:
 	auto badBinaryRegex = ctRegex!(`^0b[01]{9,}`);
 	auto badDecimalRegex = ctRegex!(`^\d{5,}`);
 }
 
 unittest
 {
-	import analysis.config;
+	import analysis.config : StaticAnalysisConfig;
 	StaticAnalysisConfig sac;
 	sac.number_style_check = true;
 	assertAnalyzerWarnings(q{

src/analysis/undocumented.d

@@ -43,16 +43,19 @@ class UndocumentedDeclarationCheck : BaseAnalyzer
 			}
 		}
 
-		bool shouldPop = false;
-		bool prevOverride = getOverride();
+		immutable bool shouldPop = dec.attributeDeclaration is null;
+		immutable bool prevOverride = getOverride();
 		bool ovr = false;
+		bool pushed = false;
 		foreach (attribute; dec.attributes)
 		{
-			shouldPop = dec.attributeDeclaration !is null;
 			if (isProtection(attribute.attribute.type))
 			{
 				if (shouldPop)
+				{
+					pushed = true;
 					push(attribute.attribute.type);
+				}
 				else
 					set(attribute.attribute.type);
 			}
@@ -62,7 +65,7 @@ class UndocumentedDeclarationCheck : BaseAnalyzer
 		if (ovr)
 			setOverride(true);
 		dec.accept(this);
-		if (shouldPop)
+		if (shouldPop && pushed)
 			pop();
 		if (ovr)
 			setOverride(prevOverride);

src/analysis/unmodified.d

@@ -108,17 +108,21 @@ class UnmodifiedFinder:BaseAnalyzer
 			dec.accept(this);
 	}
 
+	override void visit(const IdentifierChain ic)
+	{
+		if (ic.identifiers.length && interest > 0)
+			variableMightBeModified(ic.identifiers[0].text);
+		ic.accept(this);
+	}
+
 	override void visit(const IdentifierOrTemplateInstance ioti)
 	{
-//		import std.stdio : stderr;
-//		stderr.writeln(ioti.identifier.text, " ", ioti.identifier.line);
 		if (ioti.identifier != tok!"" && interest > 0)
-		{
 			variableMightBeModified(ioti.identifier.text);
-		}
 		ioti.accept(this);
 	}
 
+	mixin PartsMightModify!AsmPrimaryExp;
 	mixin PartsMightModify!IndexExpression;
 	mixin PartsMightModify!SliceExpression;
 	mixin PartsMightModify!FunctionCallExpression;

src/astprinter.d

@@ -10,13 +10,9 @@ import std.stdio;
 import std.string;
 import std.array;
 
-template tagAndAccept(string tagName)
-{
-    immutable tagAndAccept = `output.writeln("<` ~ tagName ~ `>");`
-        ~ tagName ~ `.accept(this);`
-        ~ `output.writeln("</` ~ tagName ~ `>");`;
-}
-
+/**
+ * AST visitor that outputs an XML representation of the AST to its file.
+ */
 class XMLPrinter : ASTVisitor
 {
     override void visit(const AddExpression addExpression)
@@ -95,7 +91,6 @@ class XMLPrinter : ASTVisitor
             visit(asmInstruction.identifierOrIntegerOrOpcode);
         if (asmInstruction.operands !is null)
         {
-            stderr.writeln("operands is not null");
             visit(asmInstruction.operands);
         }
         output.writeln("</asmInstruction>");
@@ -1094,6 +1089,17 @@ class XMLPrinter : ASTVisitor
         output.writeln("<ddoc>", xmlEscape(comment), "</ddoc>");
     }
 
+	/**
+	 * File that output  is written to.
+	 */
     File output;
 }
 
+private:
+
+template tagAndAccept(string tagName)
+{
+    immutable tagAndAccept = `output.writeln("<` ~ tagName ~ `>");`
+        ~ tagName ~ `.accept(this);`
+        ~ `output.writeln("</` ~ tagName ~ `>");`;
+}

src/ctags.d

@@ -14,8 +14,12 @@ import std.stdio;
 import std.array;
 import std.conv;
 
-void doNothing(string, size_t, size_t, string, bool) {}
-
+/**
+ * Prints CTAGS information to the given file.
+ * Params:
+ *     outpt = the file that CTAGS info is written to
+ *     fileNames = tags will be generated from these files
+ */
 void printCtags(File output, string[] fileNames)
 {
 	string[] tags;
@@ -41,6 +45,10 @@ void printCtags(File output, string[] fileNames)
 	tags.sort().copy(output.lockingTextWriter);
 }
 
+private:
+
+void doNothing(string, size_t, size_t, string, bool) {}
+
 class CTagsPrinter : ASTVisitor
 {
 	override void visit(const ClassDeclaration dec)

src/imports.d

@@ -9,6 +9,9 @@ import std.d.ast;
 import std.stdio;
 import std.container;
 
+/**
+ * AST visitor that collects modules imported to an R-B tree.
+ */
 class ImportPrinter : ASTVisitor
 {
 	this()
@@ -38,9 +41,11 @@ class ImportPrinter : ASTVisitor
 		imports.insert(s);
 	}
 
-	RedBlackTree!string imports;
-
 	alias visit = ASTVisitor.visit;
 
+	/// Collected imports
+	RedBlackTree!string imports;
+
+private:
 	bool ignore = true;
 }