Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .dscanner.ini
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
; Configure which static analysis checks are enabled
[analysis.config.StaticAnalysisConfig]
[dscanner.analysis.config.StaticAnalysisConfig]
; Check variable, class, struct, interface, union, and function names against t
; he Phobos style guide
style_check="disabled"
Expand Down
3 changes: 1 addition & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@
.svn/

# D Scanner binaries
dscanner
dscanner.o
/bin

# Static analysis reports
dscanner-report.json
Expand Down
4 changes: 2 additions & 2 deletions build.bat
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ set LIBDDOC=
set STDXALLOCATOR=
set STDXALLOCATORBLOCKS=

for %%x in (src\*.d) do set CORE=!CORE! %%x
for %%x in (src\analysis\*.d) do set ANALYSIS=!ANALYSIS! %%x
for %%x in (src\dscanner\*.d) do set CORE=!CORE! %%x
for %%x in (src\dscanner\analysis\*.d) do set ANALYSIS=!ANALYSIS! %%x
for %%x in (libdparse\src\dparse\*.d) do set LIBDPARSE=!LIBDPARSE! %%x
for %%x in (libdparse\src\std\experimental\*.d) do set LIBDPARSE=!LIBDPARSE! %%x
for %%x in (libddoc\src\ddoc\*.d) do set LIBDDOC=!LIBDDOC! %%x
Expand Down
6 changes: 0 additions & 6 deletions src/analysis/package.d

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.alias_syntax_check;
module dscanner.analysis.alias_syntax_check;

import dparse.ast;
import dparse.lexer;
import analysis.base;
import dscanner.analysis.base;

/**
* Checks for uses of the old alias syntax.
Expand Down Expand Up @@ -39,8 +39,8 @@ private:

unittest
{
import analysis.helpers : assertAnalyzerWarnings;
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import std.stdio : stderr;

StaticAnalysisConfig sac = disabledConfig();
Expand Down
8 changes: 4 additions & 4 deletions src/analysis/allman.d → src/dscanner/analysis/allman.d
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.allman;
module dscanner.analysis.allman;

import dparse.lexer;
import dparse.ast;
import analysis.base : BaseAnalyzer;
import dscanner.analysis.base : BaseAnalyzer;
import dsymbol.scope_ : Scope;

import std.algorithm;
Expand Down Expand Up @@ -65,8 +65,8 @@ class AllManCheck : BaseAnalyzer

unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;
import std.format : format;
import std.stdio : stderr;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,13 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.asm_style;
module dscanner.analysis.asm_style;

import std.stdio;
import dparse.ast;
import dparse.lexer;
import analysis.base;
import analysis.helpers;
import dscanner.analysis.base;
import dscanner.analysis.helpers;
import dsymbol.scope_ : Scope;

/**
Expand Down Expand Up @@ -39,7 +39,7 @@ class AsmStyleCheck : BaseAnalyzer

unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;

StaticAnalysisConfig sac = disabledConfig();
sac.asm_style_check = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.assert_without_msg;
module dscanner.analysis.assert_without_msg;

import analysis.base : BaseAnalyzer;
import dscanner.analysis.base : BaseAnalyzer;
import dsymbol.scope_ : Scope;
import dparse.lexer;
import dparse.ast;
Expand Down Expand Up @@ -92,8 +92,8 @@ unittest
{
import std.stdio : stderr;
import std.format : format;
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;

StaticAnalysisConfig sac = disabledConfig();
sac.assert_without_msg = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.auto_function;
module dscanner.analysis.auto_function;

import analysis.base;
import analysis.helpers;
import dscanner.analysis.base;
import dscanner.analysis.helpers;
import dparse.ast;
import dparse.lexer;

Expand Down Expand Up @@ -157,8 +157,8 @@ unittest
{
import std.stdio : stderr;
import std.format : format;
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;

StaticAnalysisConfig sac = disabledConfig();
sac.auto_function_check = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.auto_ref_assignment;
module dscanner.analysis.auto_ref_assignment;

import dparse.lexer;
import dparse.ast;
import analysis.base;
import dscanner.analysis.base;

/**
* Checks for assignment to auto-ref function parameters.
Expand Down Expand Up @@ -113,8 +113,8 @@ unittest
{
import std.stdio : stderr;
import std.format : format;
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;

StaticAnalysisConfig sac = disabledConfig();
sac.auto_ref_assignment_check = Check.enabled;
Expand Down
2 changes: 1 addition & 1 deletion src/analysis/base.d → src/dscanner/analysis/base.d
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
module analysis.base;
module dscanner.analysis.base;

import std.container;
import std.string;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.builtin_property_names;
module dscanner.analysis.builtin_property_names;

import std.stdio;
import std.regex;
import dparse.ast;
import dparse.lexer;
import analysis.base;
import analysis.helpers;
import dscanner.analysis.base;
import dscanner.analysis.helpers;
import dsymbol.scope_;
import std.algorithm : map;

Expand Down Expand Up @@ -102,7 +102,7 @@ private:

unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;

StaticAnalysisConfig sac = disabledConfig();
sac.builtin_property_names_check = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.comma_expression;
module dscanner.analysis.comma_expression;

import dparse.ast;
import dparse.lexer;
import analysis.base;
import dscanner.analysis.base;
import dsymbol.scope_;

/**
Expand Down
2 changes: 1 addition & 1 deletion src/analysis/config.d → src/dscanner/analysis/config.d
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.config;
module dscanner.analysis.config;

import inifiled;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
module analysis.constructors;
module dscanner.analysis.constructors;

import dparse.ast;
import dparse.lexer;
import std.stdio;
import analysis.base;
import analysis.helpers;
import dscanner.analysis.base;
import dscanner.analysis.helpers;
import dsymbol.scope_ : Scope;

class ConstructorCheck : BaseAnalyzer
Expand Down Expand Up @@ -90,7 +90,7 @@ private:

unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;

StaticAnalysisConfig sac = disabledConfig();
sac.constructor_check = Check.enabled;
Expand Down
8 changes: 4 additions & 4 deletions src/analysis/del.d → src/dscanner/analysis/del.d
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.del;
module dscanner.analysis.del;

import std.stdio;
import dparse.ast;
import dparse.lexer;
import analysis.base;
import dscanner.analysis.base;
import dsymbol.scope_;

/**
Expand All @@ -33,8 +33,8 @@ class DeleteCheck : BaseAnalyzer

unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;

StaticAnalysisConfig sac = disabledConfig();
sac.delete_check = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.duplicate_attribute;
module dscanner.analysis.duplicate_attribute;

import std.stdio;
import std.string;
import dparse.ast;
import dparse.lexer;
import analysis.base;
import analysis.helpers;
import dscanner.analysis.base;
import dscanner.analysis.helpers;
import dsymbol.scope_ : Scope;

/**
Expand Down Expand Up @@ -153,7 +153,7 @@ class DuplicateAttributeCheck : BaseAnalyzer

unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;

StaticAnalysisConfig sac = disabledConfig();
sac.duplicate_attribute = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.enumarrayliteral;
module dscanner.analysis.enumarrayliteral;

import dparse.ast;
import dparse.lexer;
import analysis.base;
import dscanner.analysis.base;
import std.algorithm : canFind, map;
import dsymbol.scope_ : Scope;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.explicitly_annotated_unittests;
module dscanner.analysis.explicitly_annotated_unittests;

import dparse.lexer;
import dparse.ast;
import analysis.base : BaseAnalyzer;
import dscanner.analysis.base : BaseAnalyzer;

import std.stdio;

Expand Down Expand Up @@ -56,8 +56,8 @@ unittest
{
import std.stdio : stderr;
import std.format : format;
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;

StaticAnalysisConfig sac = disabledConfig();
sac.explicitly_annotated_unittests = Check.enabled;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)

module analysis.final_attribute;
module dscanner.analysis.final_attribute;

import analysis.base;
import analysis.helpers;
import dscanner.analysis.base;
import dscanner.analysis.helpers;
import dparse.ast;
import dparse.lexer;

Expand Down Expand Up @@ -245,8 +245,8 @@ public:

@system unittest
{
import analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import analysis.helpers : assertAnalyzerWarnings;
import dscanner.analysis.config : StaticAnalysisConfig, Check, disabledConfig;
import dscanner.analysis.helpers : assertAnalyzerWarnings;
import std.stdio : stderr;
import std.format : format;

Expand Down
Loading