diff options
author | Sebastien Pouliot <sebastien@ximian.com> | 2010-06-05 18:50:53 +0400 |
---|---|---|
committer | Sebastien Pouliot <sebastien@ximian.com> | 2010-06-05 18:50:53 +0400 |
commit | 49902e584b0c16db3efcbb050ae5e58185b836c4 (patch) | |
tree | 77bfe06c879e95d26037a3e50ee6ecad2fc27366 | |
parent | 762c6b2a13da29c513c44df2de5ca6a44942801f (diff) |
2010-06-05 Sebastien Pouliot <sebastien@ximian.com>
* AvoidSmallNamespaceRule.cs: Use a unique NamespaceDefinition (not
a new one each time) when reporting a defect
svn path=/trunk/mono-tools/; revision=158533
-rw-r--r-- | gendarme/rules/Gendarme.Rules.Design/AvoidSmallNamespaceRule.cs | 2 | ||||
-rw-r--r-- | gendarme/rules/Gendarme.Rules.Design/ChangeLog | 5 |
2 files changed, 6 insertions, 1 deletions
diff --git a/gendarme/rules/Gendarme.Rules.Design/AvoidSmallNamespaceRule.cs b/gendarme/rules/Gendarme.Rules.Design/AvoidSmallNamespaceRule.cs index dff13682..138b556b 100644 --- a/gendarme/rules/Gendarme.Rules.Design/AvoidSmallNamespaceRule.cs +++ b/gendarme/rules/Gendarme.Rules.Design/AvoidSmallNamespaceRule.cs @@ -163,7 +163,7 @@ namespace Gendarme.Rules.Design { // don't report namespaces that are not visible outside the assembly // e.g. VS.NET adds a .Properties namespace to SWF apps if ((count > 0) && (count < Minimum)) { - NamespaceDefinition n = new NamespaceDefinition (ns); + NamespaceDefinition n = NamespaceDefinition.GetDefinition (ns); string msg = String.Format ("Only {0} visible types are defined inside this namespace.", count); // overloads of Report cannot be used here since the 'target' has been lost in the runner Runner.Report (new Defect (this, n, n, Severity.Low, Confidence.Total, msg)); diff --git a/gendarme/rules/Gendarme.Rules.Design/ChangeLog b/gendarme/rules/Gendarme.Rules.Design/ChangeLog index 2eab4dfb..c4c0faf3 100644 --- a/gendarme/rules/Gendarme.Rules.Design/ChangeLog +++ b/gendarme/rules/Gendarme.Rules.Design/ChangeLog @@ -1,3 +1,8 @@ +2010-06-05 Sebastien Pouliot <sebastien@ximian.com> + + * AvoidSmallNamespaceRule.cs: Use a unique NamespaceDefinition (not + a new one each time) when reporting a defect + 2010-05-24 Sebastien Pouliot <sebastien@ximian.com> * Makefile.am: Add define for CODE_ANALYSIS on unit tests (required |