|
Posted
over 14 years
ago
by
[email protected]
(Alex)
Hi, IMO, change your convention. This mistake is only possible in C# if foo is a boolean, but in that case you shouldn't even be doing this to begin with, but just a simple if (foo) or whatever. Regards, Alex
|
|
Posted
over 14 years
ago
by
[email protected]
(Vlad)
Hi, I have coding habits that come from C which seem to be violating some C# style guideline. I prefer writing: if (null == foo) { DoStuff(); But it seems the rule does not consider the above as good a null check as if (foo == null) Which I consider bad because it is all too easy to drop an =. C# only
|
|
Posted
over 14 years
ago
by
[email protected]
(Alexandre Victoor)
Hello Shaun I am not contributing to gendarme or any mono open source project. However, I am a user of gendarme and partcover... From my point of view you do not have to relies on gendarme to calculate complexity. OpenCover could be lighter if you just use Cecil and adapt and extract a code fragment from
|
|
Posted
over 14 years
ago
by
[email protected]
(Shaun Wilde)
Hi I am looking to add a cyclomatic complexity metric to my coverage reports from opencover (a .NET code coverage tool hosted on github). I see that Gendarme calculates a value via it's AvoidComplexMethodsRule which I can leverage. Will anyone object if I link to the gendarme library to calculate this value
|
|
Posted
over 14 years
ago
by
[email protected]
(GSS)
Hi, I download Gendarme for NANT (the task), but the download seems to miss a .dll, which is required when trying to execute the task. Where can I get the latest (and complete) download from? Thanks
|
|
Posted
over 14 years
ago
by
[email protected]
(Daniel Lidström)
Thanks Sebastien, I will file a report. /Daniel
|
|
Posted
over 14 years
ago
by
[email protected]
(Sebastien Pouliot)
Hello Daniel, I'm very busy at the moment so please fill bug reports for such issues (it's the only to ensure they won't be forgotten when I, or someone else, gets some free time ;-) Sebastien p.s. a quick guess is that 'ref' parameters are misreported when simply passed down to another call. It's probably something easy to fix if
|
|
Posted
over 14 years
ago
by
[email protected]
(Daniel Lidström)
Hello, I've just experienced a strange case of CheckParametersNullityInVisibl eMethodsRule. I don't understand why it fires for my code. Here's the "offending" code: public void SetVarValue( int lEMLArea, ref string szVar, ref string szValue, ref int lValue ) { short rc = 0;
|
|
Posted
over 14 years
ago
by
[email protected]
(Andrew Hanson)
Just to throw my hat in the ring, I've been using Gendarme pretty heavily for a few months now on multiple VB projects. It's not always perfect, but I can certainly try to help you out if Gendarme is acting funky and you're not quite sure what's up. Great product btw, and so happy to have it around
|
|
Posted
over 14 years
ago
by
[email protected]
(Daniel Jelinski)
In asp.net generated files (Page.designer.cs) there are multiple protected fields. It would be hard to convert them to private. AvoidVisibleFields rule reports them as violations. Its FxCop counterpart, DoNotDeclareVisibleInstanceFie lds does not report them. It would be convenient if Gendarme skipped them too.
|