Fix indentation
This commit is contained in:
Hackerpilot
parent
cb2ba1929c
commit
e4426081dd
|
|
@ -205,10 +205,14 @@ MessageSet analyze(string fileName, const Module m, const StaticAnalysisConfig a
|
|||
return null;
|
||||
|
||||
auto symbolAllocator = scoped!ASTAllocator();
|
||||
version (unittest) enum ut = true; else enum ut = false;
|
||||
version (unittest)
|
||||
enum ut = true;
|
||||
else
|
||||
enum ut = false;
|
||||
|
||||
|
||||
auto first = scoped!FirstPass(m, internString(fileName), symbolAllocator,
|
||||
symbolAllocator, true, &moduleCache, null);
|
||||
symbolAllocator, true, &moduleCache, null);
|
||||
first.run();
|
||||
|
||||
secondPass(first.rootSymbol, first.moduleScope, moduleCache);
|
||||
|
|
@ -220,140 +224,140 @@ MessageSet analyze(string fileName, const Module m, const StaticAnalysisConfig a
|
|||
|
||||
if (analysisConfig.asm_style_check != Check.disabled)
|
||||
checks ~= new AsmStyleCheck(fileName, moduleScope,
|
||||
analysisConfig.asm_style_check == Check.skipTests && !ut);
|
||||
analysisConfig.asm_style_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.backwards_range_check != Check.disabled)
|
||||
checks ~= new BackwardsRangeCheck(fileName, moduleScope,
|
||||
analysisConfig.backwards_range_check == Check.skipTests && !ut);
|
||||
analysisConfig.backwards_range_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.builtin_property_names_check != Check.disabled)
|
||||
checks ~= new BuiltinPropertyNameCheck(fileName, moduleScope,
|
||||
analysisConfig.builtin_property_names_check == Check.skipTests && !ut);
|
||||
analysisConfig.builtin_property_names_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.comma_expression_check != Check.disabled)
|
||||
checks ~= new CommaExpressionCheck(fileName, moduleScope,
|
||||
analysisConfig.comma_expression_check == Check.skipTests && !ut);
|
||||
analysisConfig.comma_expression_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.constructor_check != Check.disabled)
|
||||
checks ~= new ConstructorCheck(fileName, moduleScope,
|
||||
analysisConfig.constructor_check == Check.skipTests && !ut);
|
||||
analysisConfig.constructor_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.could_be_immutable_check != Check.disabled)
|
||||
checks ~= new UnmodifiedFinder(fileName, moduleScope,
|
||||
analysisConfig.could_be_immutable_check == Check.skipTests && !ut);
|
||||
analysisConfig.could_be_immutable_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.delete_check != Check.disabled)
|
||||
checks ~= new DeleteCheck(fileName, moduleScope,
|
||||
analysisConfig.delete_check == Check.skipTests && !ut);
|
||||
analysisConfig.delete_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.duplicate_attribute != Check.disabled)
|
||||
checks ~= new DuplicateAttributeCheck(fileName, moduleScope,
|
||||
analysisConfig.duplicate_attribute == Check.skipTests && !ut);
|
||||
analysisConfig.duplicate_attribute == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.enum_array_literal_check != Check.disabled)
|
||||
checks ~= new EnumArrayLiteralCheck(fileName, moduleScope,
|
||||
analysisConfig.enum_array_literal_check == Check.skipTests && !ut);
|
||||
analysisConfig.enum_array_literal_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.exception_check != Check.disabled)
|
||||
checks ~= new PokemonExceptionCheck(fileName, moduleScope,
|
||||
analysisConfig.exception_check == Check.skipTests && !ut);
|
||||
analysisConfig.exception_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.float_operator_check != Check.disabled)
|
||||
checks ~= new FloatOperatorCheck(fileName, moduleScope,
|
||||
analysisConfig.float_operator_check == Check.skipTests && !ut);
|
||||
analysisConfig.float_operator_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.function_attribute_check != Check.disabled)
|
||||
checks ~= new FunctionAttributeCheck(fileName, moduleScope,
|
||||
analysisConfig.function_attribute_check == Check.skipTests && !ut);
|
||||
analysisConfig.function_attribute_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.if_else_same_check != Check.disabled)
|
||||
checks ~= new IfElseSameCheck(fileName, moduleScope,
|
||||
analysisConfig.if_else_same_check == Check.skipTests&& !ut);
|
||||
analysisConfig.if_else_same_check == Check.skipTests&& !ut);
|
||||
|
||||
if (analysisConfig.label_var_same_name_check != Check.disabled)
|
||||
checks ~= new LabelVarNameCheck(fileName, moduleScope,
|
||||
analysisConfig.label_var_same_name_check == Check.skipTests && !ut);
|
||||
analysisConfig.label_var_same_name_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.length_subtraction_check != Check.disabled)
|
||||
checks ~= new LengthSubtractionCheck(fileName, moduleScope,
|
||||
analysisConfig.length_subtraction_check == Check.skipTests && !ut);
|
||||
analysisConfig.length_subtraction_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.local_import_check != Check.disabled)
|
||||
checks ~= new LocalImportCheck(fileName, moduleScope,
|
||||
analysisConfig.local_import_check == Check.skipTests && !ut);
|
||||
analysisConfig.local_import_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.logical_precedence_check != Check.disabled)
|
||||
checks ~= new LogicPrecedenceCheck(fileName, moduleScope,
|
||||
analysisConfig.logical_precedence_check == Check.skipTests && !ut);
|
||||
analysisConfig.logical_precedence_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.mismatched_args_check != Check.disabled)
|
||||
checks ~= new MismatchedArgumentCheck(fileName, moduleScope,
|
||||
analysisConfig.mismatched_args_check == Check.skipTests && !ut);
|
||||
analysisConfig.mismatched_args_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.number_style_check != Check.disabled)
|
||||
checks ~= new NumberStyleCheck(fileName, moduleScope,
|
||||
analysisConfig.number_style_check == Check.skipTests && !ut);
|
||||
analysisConfig.number_style_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.object_const_check != Check.disabled)
|
||||
checks ~= new ObjectConstCheck(fileName, moduleScope,
|
||||
analysisConfig.object_const_check == Check.skipTests && !ut);
|
||||
analysisConfig.object_const_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.opequals_tohash_check != Check.disabled)
|
||||
checks ~= new OpEqualsWithoutToHashCheck(fileName, moduleScope,
|
||||
analysisConfig.opequals_tohash_check == Check.skipTests && !ut);
|
||||
analysisConfig.opequals_tohash_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.redundant_parens_check != Check.disabled)
|
||||
checks ~= new RedundantParenCheck(fileName, moduleScope,
|
||||
analysisConfig.redundant_parens_check == Check.skipTests && !ut);
|
||||
analysisConfig.redundant_parens_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.style_check != Check.disabled)
|
||||
checks ~= new StyleChecker(fileName, moduleScope,
|
||||
analysisConfig.style_check == Check.skipTests && !ut);
|
||||
analysisConfig.style_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.undocumented_declaration_check != Check.disabled)
|
||||
checks ~= new UndocumentedDeclarationCheck(fileName, moduleScope,
|
||||
analysisConfig.undocumented_declaration_check == Check.skipTests && !ut);
|
||||
analysisConfig.undocumented_declaration_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.unused_label_check != Check.disabled)
|
||||
checks ~= new UnusedLabelCheck(fileName, moduleScope,
|
||||
analysisConfig.unused_label_check == Check.skipTests && !ut);
|
||||
analysisConfig.unused_label_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.unused_variable_check != Check.disabled)
|
||||
checks ~= new UnusedVariableCheck(fileName, moduleScope,
|
||||
analysisConfig.unused_variable_check == Check.skipTests && !ut);
|
||||
analysisConfig.unused_variable_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.long_line_check != Check.disabled)
|
||||
checks ~= new LineLengthCheck(fileName, tokens,
|
||||
analysisConfig.long_line_check == Check.skipTests && !ut);
|
||||
analysisConfig.long_line_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.auto_ref_assignment_check != Check.disabled)
|
||||
checks ~= new AutoRefAssignmentCheck(fileName,
|
||||
analysisConfig.auto_ref_assignment_check == Check.skipTests && !ut);
|
||||
analysisConfig.auto_ref_assignment_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.incorrect_infinite_range_check != Check.disabled)
|
||||
checks ~= new IncorrectInfiniteRangeCheck(fileName,
|
||||
analysisConfig.incorrect_infinite_range_check == Check.skipTests && !ut);
|
||||
analysisConfig.incorrect_infinite_range_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.useless_assert_check != Check.disabled)
|
||||
checks ~= new UselessAssertCheck(fileName,
|
||||
analysisConfig.useless_assert_check == Check.skipTests && !ut);
|
||||
analysisConfig.useless_assert_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.alias_syntax_check != Check.disabled)
|
||||
checks ~= new AliasSyntaxCheck(fileName,
|
||||
analysisConfig.alias_syntax_check == Check.skipTests && !ut);
|
||||
analysisConfig.alias_syntax_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.static_if_else_check != Check.disabled)
|
||||
checks ~= new StaticIfElse(fileName,
|
||||
analysisConfig.static_if_else_check == Check.skipTests && !ut);
|
||||
analysisConfig.static_if_else_check == Check.skipTests && !ut);
|
||||
|
||||
if (analysisConfig.lambda_return_check != Check.disabled)
|
||||
if (analysisConfig.lambda_return_check != Check.disabled)
|
||||
checks ~= new LambdaReturnCheck(fileName,
|
||||
analysisConfig.lambda_return_check == Check.skipTests && !ut);
|
||||
analysisConfig.lambda_return_check == Check.skipTests && !ut);
|
||||
|
||||
version (none)
|
||||
if (analysisConfig.redundant_if_check != Check.disabled)
|
||||
checks ~= new IfStatementCheck(fileName, moduleScope,
|
||||
analysisConfig.redundant_if_check == Check.skipTests && !ut);
|
||||
analysisConfig.redundant_if_check == Check.skipTests && !ut);
|
||||
|
||||
foreach (check; checks)
|
||||
{
|
||||
|
|
|
|||
Loading…
Reference in New Issue