diff options
| author | Sebastien Pouliot <sebastien@ximian.com> | 2008-02-12 21:28:00 +0300 |
|---|---|---|
| committer | Sebastien Pouliot <sebastien@ximian.com> | 2008-02-12 21:28:00 +0300 |
| commit | f24e4e7abd71a15a50ddbf700d312ac824d46e33 (patch) | |
| tree | cb5b50769c95a5f3eee469c2defdd246cc33a77b /gendarme/rules | |
| parent | 302d843f91710ae3042ac98d16b3aa14f88ee63e (diff) | |
2008-02-12 Sebastien Pouliot <sebastien@ximian.com>
* Gendarme.Rules.Concurrency.xml.in: Fix copy-paste mistake for
NonConstantStaticFieldsShouldNotBeVisibleRule
svn path=/trunk/mono-tools/; revision=95541
Diffstat (limited to 'gendarme/rules')
| -rw-r--r-- | gendarme/rules/Gendarme.Rules.Concurrency/ChangeLog | 5 | ||||
| -rw-r--r-- | gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.xml.in | 4 |
2 files changed, 7 insertions, 2 deletions
diff --git a/gendarme/rules/Gendarme.Rules.Concurrency/ChangeLog b/gendarme/rules/Gendarme.Rules.Concurrency/ChangeLog index daecf9c0..a503ce45 100644 --- a/gendarme/rules/Gendarme.Rules.Concurrency/ChangeLog +++ b/gendarme/rules/Gendarme.Rules.Concurrency/ChangeLog @@ -1,3 +1,8 @@ +2008-02-12 Sebastien Pouliot <sebastien@ximian.com> + + * Gendarme.Rules.Concurrency.xml.in: Fix copy-paste mistake for + NonConstantStaticFieldsShouldNotBeVisibleRule + 2008-01-21 Sebastien Pouliot <sebastien@ximian.com> * NonConstantStaticFieldsShouldNotBeVisibleRule.cs: New. Rule that diff --git a/gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.xml.in b/gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.xml.in index c5a6c486..30693edf 100644 --- a/gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.xml.in +++ b/gendarme/rules/Gendarme.Rules.Concurrency/Gendarme.Rules.Concurrency.xml.in @@ -3,13 +3,13 @@ Type="Gendarme.Rules.Concurrency.DoubleCheckLockingRule, Gendarme.Rules.Concurrency, Version=@GENDARME_VERSION@, Culture=neutral, PublicKeyToken=null" Uri="http://www.mono-project.com/Gendarme.Rules.Concurrency#DoubleCheckLockingRule" > <problem>Method '{0}' uses the unreliable double-check locking technique.</problem> - <solution>Remove the lock check that occurs outside of the protected region. Or see: http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html</solution> + <solution>Remove the lock check that occurs outside of the protected region. See: http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html</solution> </rule> <rule Name="NonConstantStaticFieldsShouldNotBeVisibleRule" Type="Gendarme.Rules.Concurrency.NonConstantStaticFieldsShouldNotBeVisibleRule, Gendarme.Rules.Concurrency, Version=@GENDARME_VERSION@, Culture=neutral, PublicKeyToken=null" Uri="http://www.mono-project.com/Gendarme.Rules.Concurrency#NonConstantStaticFieldsShouldNotBeVisible" > <problem>Type '{0}' has some static fields that are not constant. They may represent problems in multithreaded applications.</problem> - <solution>Remove the lock check that occurs outside of the protected region. Or see: http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html</solution> + <solution>Change the field to read-only, mark it [ThreadStatic] or make it non visible outside the assembly.</solution> </rule> <rule Name="WriteStaticFieldFromInstanceMethodRule" Type="Gendarme.Rules.Concurrency.WriteStaticFieldFromInstanceMethodRule, Gendarme.Rules.Concurrency, Version=@GENDARME_VERSION@, Culture=neutral, PublicKeyToken=null" |