Age | Commit message (Collapse) | Author |
|
|
|
2010-06-10 Sebastien Pouliot <sebastien@ximian.com>
* AvoidLargeClassesTest.cs: Adjust defects count for prefixes
In .:
2010-06-10 Sebastien Pouliot <sebastien@ximian.com>
* AvoidLargeClassesRule.cs: Be more accurate when reporting. E.g.
report all common prefixes with a count
svn path=/trunk/mono-tools/; revision=158786
|
|
2010-06-10 Sebastien Pouliot <sebastien@ximian.com>
* AvoidCodeDuplicatedInSameClassTest.cs: Add test case for
a (fixed) false positive when compiled with CSC
In .:
2010-06-10 Sebastien Pouliot <sebastien@ximian.com>
* AvoidLongParameterListsRule.cs: Move HasConstructor check before
a call. Rename two methods (Smaller->Smallest)
* CodeDuplicatedLocator.cs: Don't create more than one empty List
of patterns and avoid some allocations. Saves 74kb on self-test
* Pattern.cs: Reuse the instructions array when possible. Saves
1320kb while executing self-test
svn path=/trunk/mono-tools/; revision=158784
|
|
svn path=/trunk/mono-tools/; revision=158747
|
|
2010-01-26 Sebastien Pouliot <sebastien@ximian.com>
* AvoidMessageChainsTest.cs: Adjust test to new condition and
values.
In .:
2010-04-26 Sebastien Pouliot <sebastien@ximian.com>
* AvoidMessageChainsRule.cs: Change MaxChainLength from 5 to 4
and condition from >= to > (same effect) so the rule description
(and documentation) makes more sense.
svn path=/trunk/mono-tools/; revision=156153
|
|
svn path=/trunk/mono-tools/; revision=150176
|
|
svn path=/trunk/mono-tools/; revision=150094
|
|
svn path=/trunk/mono-tools/; revision=122265
|
|
* AvoidSwitchStatementsTest.cs: Added a test for table-driven
comparisons, by example with strings.
svn path=/trunk/mono-tools/; revision=122010
|
|
* AvoidSwitchStatementsTest.cs: New. Unit tests for
AvoidSwitchStatementsRule.
svn path=/trunk/mono-tools/; revision=122000
|
|
when build on windows
svn path=/trunk/mono-tools/; revision=121758
|
|
* AvoidCodeDuplicatedInSameClassTest.cs: Fix a test case so it
compiles with CSC (mcs bug #460016)
svn path=/trunk/mono-tools/; revision=121756
|
|
* AvoidCodeDuplicatedInSameClassTest.cs: Added more tests in order to
improve the confidence about the code duplicated rules.
svn path=/trunk/mono-tools/; revision=121703
|
|
* AvoidLargeClassesTest.cs: Test for DoesNotApply if a type has no
fields.
svn path=/trunk/mono-tools/; revision=121566
|
|
Mono.Cecil.Pdb and the installer projects
svn path=/trunk/mono-tools/; revision=120956
|
|
svn path=/trunk/mono-tools/; revision=118338
|
|
svn path=/trunk/mono-tools/; revision=118322
|
|
* gendarme/rules/rules.mds:
* gendarme/rules/common.make:
* gendarme/rules/Makefile.am:
* gendarme/console/gendarme.csproj:
* gendarme/rules/Test.Rules/Test.Rules.csproj:
* gendarme/framework/Gendarme.Framework.csproj:
* gendarme/rules/Gendarme.Rules.Ui/Makefile.am:
* gendarme/framework/Test/Tests.Framework.csproj:
* gendarme/rules/Gendarme.Rules.Naming/Makefile.am:
* gendarme/rules/Gendarme.Rules.Smells/Makefile.am:
* gendarme/rules/Gendarme.Rules.Design/Makefile.am:
* gendarme/rules/Gendarme.Rules.Security/Makefile.am:
* gendarme/swf-wizard-runner/swf-wizard-runner.csproj:
* gendarme/rules/Gendarme.Rules.Exceptions/Makefile.am:
* gendarme/rules/Gendarme.Rules.Concurrency/Makefile.am:
* gendarme/rules/Gendarme.Rules.BadPractice/Makefile.am:
* gendarme/rules/Gendarme.Rules.Portability/Makefile.am:
* gendarme/rules/Gendarme.Rules.Performance/Makefile.am:
* gendarme/rules/Gendarme.Rules.Correctness/Makefile.am:
* gendarme/rules/Gendarme.Rules.Design.Linq/Makefile.am:
* gendarme/rules/Gendarme.Rules.Ui/Gendarme.Rules.Ui.mdp:
* gendarme/rules/Gendarme.Rules.Security.Cas/Makefile.am:
* gendarme/rules/Gendarme.Rules.Serialization/Makefile.am:
* gendarme/rules/Gendarme.Rules.Design.Generic/Makefile.am:
* gendarme/rules/Gendarme.Rules.Maintainability/Makefile.am:
* gendarme/rules/Gendarme.Rules.Ui/Gendarme.Rules.Ui.csproj:
* gendarme/rules/Gendarme.Rules.Interoperability/Makefile.am:
* gendarme/rules/Gendarme.Rules.Ui/Test/Tests.Rules.Ui.csproj:
* gendarme/rules/Gendarme.Rules.Smells/Gendarme.Rules.Smells.mdp:
* gendarme/rules/Gendarme.Rules.Naming/Gendarme.Rules.Naming.mdp:
* gendarme/rules/Gendarme.Rules.Design/Gendarme.Rules.Design.mdp:
* gendarme/rules/Gendarme.Rules.Smells/Gendarme.Rules.Smells.csproj:
* gendarme/rules/Gendarme.Rules.Naming/Gendarme.Rules.Naming.csproj:
* gendarme/rules/Gendarme.Rules.Design/Gendarme.Rules.Design.csproj:
* gendarme/rules/Gendarme.Rules.Security/Gendarme.Rules.Security.mdp:
* gendarme/rules/Gendarme.Rules.Smells/Test/Tests.Rules.Smells.csproj:
* gendarme/rules/Gendarme.Rules.Naming/Test/Tests.Rules.Naming.csproj:
* gendarme/rules/Gendarme.Rules.Design/Test/Tests.Rules.Design.csproj:
* gendarme/rules/Gendarme.Rules.Security/Gendarme.Rules.Security.csproj:
* gendarme/rules/Gendarme.Rules.Exceptions/Gendarme.Rules.Exceptions.mdp:
* gendarme/rules/Gendarme.Rules.Security/Test/Tests.Rules.Security.csproj:
* gendarme/rules/Gendarme.Rules.BadPractice/Gendarme.Rules.BadPractice.mdp:
* gendarme/rules/Gendarme.Rules.Correctness/Gendarme.Rules.Correctness.mdp:
* gendarme/rules/Gendarme.Rules.Design.Linq/Gendarme.Rules.Design.Linq.mdp:
* gendarme/rules/Gendarme.Rules.Portability/Gendarme.Rules.Portability.mdp:
* gendarme/rules/Gendarme.Rules.Performance/Gendarme.Rules.Performance.mdp:
* gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.mdp:
* gendarme/rules/Gendarme.Rules.Exceptions/Gendarme.Rules.Exceptions.csproj:
* gendarme/rules/Gendarme.Rules.Security.Cas/Gendarme.Rules.Security.Cas.mdp:
* gendarme/rules/Gendarme.Rules.Exceptions/Test/Tests.Rules.Exceptions.csproj:
* gendarme/rules/Gendarme.Rules.BadPractice/Gendarme.Rules.BadPractice.csproj:
* gendarme/rules/Gendarme.Rules.Design.Linq/Gendarme.Rules.Design.Linq.csproj:
* gendarme/rules/Gendarme.Rules.Performance/Gendarme.Rules.Performance.csproj:
* gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.csproj:
* gendarme/rules/Gendarme.Rules.Correctness/Gendarme.Rules.Correctness.csproj:
* gendarme/rules/Gendarme.Rules.Portability/Gendarme.Rules.Portability.csproj:
* gendarme/rules/Gendarme.Rules.Serialization/Gendarme.Rules.Serialization.mdp:
* gendarme/rules/Gendarme.Rules.Security.Cas/Gendarme.Rules.Security.Cas.csproj:
* gendarme/rules/Gendarme.Rules.Correctness/Test/Tests.Rules.Correctness.csproj:
* gendarme/rules/Gendarme.Rules.BadPractice/Test/Tests.Rules.BadPractice.csproj:
* gendarme/rules/Gendarme.Rules.Portability/Test/Tests.Rules.Portability.csproj:
* gendarme/rules/Gendarme.Rules.Performance/Test/Tests.Rules.Performance.csproj:
* gendarme/rules/Gendarme.Rules.Concurrency/Test/Tests.Rules.Concurrency.csproj:
* gendarme/rules/Gendarme.Rules.Design.Generic/Gendarme.Rules.Design.Generic.mdp:
* gendarme/rules/Gendarme.Rules.Serialization/Gendarme.Rules.Serialization.csproj:
* gendarme/rules/Gendarme.Rules.Maintainability/Gendarme.Rules.Maintainability.mdp:
* gendarme/rules/Gendarme.Rules.Serialization/Test/Tests.Rules.Serialization.csproj:
* gendarme/rules/Gendarme.Rules.Design.Generic/Gendarme.Rules.Design.Generic.csproj:
* gendarme/rules/Gendarme.Rules.Interoperability/Gendarme.Rules.Interoperability.mdp:
* gendarme/rules/Gendarme.Rules.Maintainability/Gendarme.Rules.Maintainability.csproj:
* gendarme/rules/Gendarme.Rules.Interoperability/Gendarme.Rules.Interoperability.csproj:
* gendarme/rules/Gendarme.Rules.Maintainability/Test/Tests.Rules.Maintainability.csproj:
* gendarme/rules/Gendarme.Rules.Interoperability/Test/Tests.Rules.Interoperability.csproj:
Activate makefile integration for all the rules projects. Remove the
old mdp files as they are no longer needed.
svn path=/trunk/mono-tools/; revision=118184
|
|
* AvoidMessageChainsTest.cs: Added two test cases 1) verify that we
work if distinct chains are not separated by a non-call 2) verify
that a relatively complex linq query does not fail.
svn path=/trunk/mono-tools/; revision=117677
|
|
svn path=/trunk/mono-tools/; revision=114457
|
|
* AvoidCodeDuplicatedInSameClassTest.cs: Added test for the issue that
Alan notified to the group.
svn path=/trunk/mono-tools/; revision=113066
|
|
* AvoidMessageChainsTest.cs: Re-enable some tests and add new ones
for better coverage.
* Tests.Rules.Smells.csproj: Add tests to MD project file.
svn path=/trunk/mono-tools/; revision=112717
|
|
svn path=/trunk/mono-tools/; revision=111623
|
|
svn path=/trunk/mono-tools/; revision=111622
|
|
* AvoidSpeculativeGeneralityTest.cs: Update unit tests to the new
syntax.
* Tests.Rules.Smells.csproj: Add a reference to Mono.Cecil.Pdb.dll
to make sure it's available when running unit tests.
svn path=/trunk/mono-tools/; revision=110546
|
|
* AvoidLongMethodsTest.cs: Update unit tests to the new syntax.
Test cases are now reused in both the (old) IL-approximation mode
and the (new) logical source line of code modes.
svn path=/trunk/mono-tools/; revision=110330
|
|
* AvoidLongMethodsTest.cs: Add test case for a class with readonly
fields.
svn path=/trunk/mono-tools/; revision=108412
|
|
* AvoidLongMethodsTest.cs: Add test case for non-static constructors.
svn path=/trunk/mono-tools/; revision=108251
|
|
* AvoidLongParameterListsTest.cs: Add test case for MulticastDelegate
svn path=/trunk/mono-tools/; revision=108102
|
|
svn path=/trunk/mono-tools/; revision=107924
|
|
rules/Gendarme.Rules.Ui/Test/Test.Rules.Ui.mdp,
rules/Gendarme.Rules.Serialization/Test/Test.Rules.Serialization.mdp,
rules/Gendarme.Rules.Exceptions/Gendarme.Rules.Exceptions.mdp,
rules/Gendarme.Rules.Interoperability/Test/Test.Rules.Interoperability.mdp,
rules/Gendarme.Rules.Correctness/Test/Test.Rules.Correctness.mdp,
rules/Gendarme.Rules.Concurrency/Test/Test.Rules.Concurrency.mdp,
rules/Gendarme.Rules.Smells/Test/Test.Rules.Smells.mdp,
rules/Gendarme.Rules.Performance/Test/Test.Rules.Performance.mdp,
rules/Gendarme.Rules.Portability/Test/Test.Rules.Portability.mdp:
Fix references.
svn path=/trunk/mono-tools/; revision=102958
|
|
* AvoidLargeClassesTest.cs, AvoidSpeculativeGeneralityTest.cs,
AvoidCodeDuplicatedInSameClassTest.cs, AvoidLongMethodsTest.cs,
AvoidCodeDuplicatedInSiblingClassesTest.cs: The reference for TestRunner
class has been changed, it is placed now in Test.Rules.Helpers
namespace.
svn path=/trunk/mono-tools/; revision=102927
|
|
* AvoidLongParameterListsTest.cs: Add test case with a static ctor
and a long instance ctor.
svn path=/trunk/mono-tools/; revision=102684
|
|
svn path=/trunk/mono-tools/; revision=101959
|
|
* AvoidMessageChainsRule.cs: Trimming down several false positives.
* Test/AviodMessageChainsTest.cs: Replaced tests with better ones. The
latest gives us a lot of false positives.
svn path=/trunk/mono-tools/; revision=101943
|
|
* AvoidMessageChainsRule.cs: Implemented logic for detect chains created
with local variables. By the moment it's a bit constrained, only 5
local variables and without reassignments.
* Test/AvoidMessageChainsTest.cs: Added test for check chains created
with local variables.
svn path=/trunk/mono-tools/; revision=101931
|
|
* AvoidMessageChainsTest.cs: Added tests for check chains in the
arguments.
svn path=/trunk/mono-tools/; revision=101913
|
|
* AvoidMessageChainsRule.cs: New. First rule in the Rule Day :)
* Test/AvoidMessageChainsTest.cs: New. Tests for AvoidMessageChains
rule.
* Makefile.am: Added both to the build.
svn path=/trunk/mono-tools/; revision=101907
|
|
* AvoidLongParameterListsRule.cs: Implemented delegate checking.
* Test/AvoidLongParameterListsRule.cs: Added tests for delegate special
checking.
svn path=/trunk/mono-tools/; revision=101110
|
|
* AvoidLongParameterListsRule.cs: Refactored to be a ITypeRule instead
of IMethodRule.
* Test/AvoidLongParameterListsRule.cs: Updated tests for the new
refactoring and updated with the new Test utilites.
* Makefile.am: Added the Test.Smells reference.
svn path=/trunk/mono-tools/; revision=101107
|
|
svn path=/trunk/mono-tools/; revision=100376
|
|
/cecil/lib
svn path=/trunk/mono-tools/; revision=98649
|
|
extension methods
svn path=/trunk/mono-tools/; revision=97467
|
|
svn path=/trunk/mono-tools/; revision=95902
|
|
* Test/*.cs: Refactor the tests for accept the new rule's format.
* *.cs: Adapt the rules to the new framework.
* Makefile.am: Removed the -langversion:linq option. You should use at
least the 1.9 gmcs version to compile.
svn path=/trunk/mono-tools/; revision=95774
|
|
svn path=/trunk/mono-tools/; revision=95408
|
|
* AvoidLargeClassesRule.cs: Avoid positive when a class contains
constants or readonly fields.
* Test/AvoidLargeClassesTest.cs: Added a test for class storing
common constant values.
svn path=/trunk/mono-tools/; revision=95169
|
|
* AvoidLongMethodsRule.cs: The rule should be applied to static
constructors. Now we add several extra instructions for
each field for initialize, specially if the field is an array.
* Test/AvoidLongMethodsTest.cs: Added a test for the new strategy for
static constructors.
svn path=/trunk/mono-tools/; revision=95150
|
|
* AvoidLongMethods.cs: The rule shouldn't be applied to static
constructors.
* AvoidLongMethodsTest.cs: Added a test for static constructors. This
rule shouldn't be applied to static constructors.
svn path=/trunk/mono-tools/; revision=95061
|
|
svn path=/trunk/mono-tools/; revision=94478
|