News
Re: [gendarme] NullReferenceException
Hello Markus,
Could you try the gendarme console runner ? i.e. gendarme.exe in a
console window. This should give us a few more details including, iirc,
the method where the NRE occurred.
If you can identity the method then it should be simple to extract a
test case (without disclosing the DLL or the IP within).
Re: NullReferenceException
Hi,
[...]
Additional information: Running GendarmeWizard in Mono seems to freeze
the UI without any interaction after finding of 90 problems. Running
gendarme from command line via mono (Mono 2.4 on the same windows
vista)
NullReferenceException
Hello,
We're currently experimenting with Code Analysis tools, and I tried
Gendarme on one of our DLLs (proprietary code, so I'm not sure whether
I get permission to disclose that dll). I Installed the Gendarme
Package and started
Re: [gendarme] Re: Review request for new rule: DelegatesPassedToNativeCodeMustIncludeExceptionHandlingRule
On Tue, Feb 17, 2009 at 10:26 AM, Rolf Bjarne Kvinge
Actually a stfld to 'this' is not necessarily safe, it depends on how
the method was called - callvirt does a nullcheck when calling, while
call doesn't (VB and C# compilers will always insert a callvirt when
an instance method is called, so this isn't necessarily true. This
Re: [gendarme] Re: Review request for new rule: DelegatesPassedToNativeCodeMustIncludeExceptionHandlingRule
Hi Sebastien
My time to say sorry for the delay now ;-)
The wiki has history, so this wasn't a problem.
At the time I was fooled by the fact that nothing was written to the
console, now that I've found the html report I have fixed
Re: UsePreferredTermsRule - Cant Can't Cannot
Yep this is a problem - sadly solving it introduce new ones, e.g.
CANT. The rule checks for 'terms' not the "correct" casing (i.e. it
does not know about other, standard or custom, rule that check for
casing). I added a note to review this [1] - we should at least be
able to reduce the confidence level in most cases.
Re: Shared code analysis annotations
Hello Steve,
Thanks :)
Right. The current code only deals with a "method-by-method" basis.
A common set would be nice. However I'm putting my (personal ;-) faith
in code contracts.
I'm not a Resharper user, actually
UsePreferredTermsRule - Cant Can't Cannot
Hi,
I've got this rule:
Target: System.Boolean Foo.MessageStatus::CanTransiti onTo
(Foo.MessageStatus)
Assembly: Foo, Version=1.0.1.0, Culture=neutral, PublicKeyToken=null
Severity: Medium Confidence: High
Source: C:\Foo\MessageStatus.cs(≈63)
Details: Term 'Cant' should be replaced with 'Cannot'.
Shared code analysis annotations
Hi,
Love Gendarme, great work!
I've found that it's giving me false positives, epecially for the rule
'CheckParametersNullityInVisib leMethodsRule'.
To be fair, Gendarme is correct in what it's reporting as I wrap 'if
(arg==null) throw .... in a wrapper. e.g.
public XmlWrapper(XNode xml)
Re: [gendarme] ignoring false positives
At some point we'll support the [SuppressMessage] attribute that MS use
to ignore defects with FxCop. About 50% of the code is already there.
In the mean time the solution (which, personally, I much prefer to using
attributes) is to