Gendarme Google Group

Gendarme 2.6 Windows Installer (and ZIPped binaries)

sebastien.poul...@gmail.com (Sebastien Pouliot) — Tue, 05 Jan 2010 18:46:35 UT

After a bit of delay [1] here are the files [2] for non-linux [3]
users.
Like I said for the preview, the win32 installer is new (not based on
the previous installers) so you either have to remove previous
Gendarme versions *or* you can keep them in parallel (2.6 will install
in another directory so both will

Re: [gendarme] AvoidUninstantiatedInternalClassesRule

jesjo...@mindspring.com (Jesse Jones) — Fri, 01 Jan 2010 21:33:18 UT

I've gone back to work, and barely have time for my own projects let alone gendarme. :-)
-- Jesse

Re: [gendarme] AvoidUninstantiatedInternalClassesRule

sebastien.poul...@gmail.com (Sebastien Pouliot) — Fri, 01 Jan 2010 21:18:08 UT

Good point, documentation is more important than some extra, unused,
metadata. Patch w/unit test are welcome!
Happy New Year,
Sebastien

AvoidUninstantiatedInternalClassesRule

jesjo...@mindspring.com (Jesse Jones) — Fri, 01 Jan 2010 02:18:42 UT

This rule should not fire for types named "NamespaceDoc" because documentation tools tend to use that for documenting namespaces. For example:
namespace Mono.Options
{
/// <summary>This namespace contains the command-line options processing library distributed by the mono team.</summary>
internal sealed class NamespaceDoc

Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Sat, 12 Dec 2009 18:24:28 UT

It's unlikely that such code will be found (often) so it does not look
like it's worth the extra complexity (for both the rule maintenance
and the extra time it would require to "normal" code).

Yes. Please add a FIXME there. I'm looking into adding "analysis
warnings" (for 2.8) and it could be a case where the rule could report

Re: [gendarme] Re: Gendarme 2.6 Preview 1 is ready for download

rolfbja...@gmail.com (Rolf Bjarne Kvinge) — Fri, 11 Dec 2009 23:47:24 UT

I had a look, and the code is using branches to choose different ways
to use what's on the stack, and since the rule doesn't bother with the
branches, it ends up with a negative stack size since it evaluated all
ways the stack was used, in pseudo code something like this:

load int into stack location 1

Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Fri, 11 Dec 2009 03:00:47 UT

Thanks, I got it now.

I'll try to get Rolf to fix it on the above first :-) IKVM is huge
and, like I said, generated code is not a good target for Gendarme
(but it would be for Cecil).

It's not as automatic as I would like. Mono is upgrading it's bots so
we should be able to get the tests run at each build. However the

Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Fri, 11 Dec 2009 02:52:39 UT

On Dec 10, 7:54 am, Sebastien Pouliot <sebastien.poul...@gmail.com>
wrote:

Ok, fixed in SVN. It looks like we already ignored most error but the
new CCI implementation of the PDB reader use InvalidOperationException
(while the old COM based one did not). In any case the "warnings" are
something to re-work more globally in the future.

Re: Gendarme 2.6 Preview 1 is ready for download

m...@soloplan.de (MarkusSchaber) — Thu, 10 Dec 2009 14:30:26 UT

Hi, Sebastien,

On 10 Dez., 02:59, Sebastien Pouliot <sebastien.poul...@gmail.com>
wrote:

Yes, indeed. In case you maintain some automatically running test
suite (unit tests etc.), you should add whatever free software DLL
for .NET you can get. :-)

Markus

Re: Gendarme 2.6 Preview 1 is ready for download

m...@soloplan.de (MarkusSchaber) — Thu, 10 Dec 2009 14:28:45 UT

Hi, Sebastien,

On 10 Dez., 02:48, Sebastien Pouliot <sebastien.poul...@gmail.com>
wrote:

I did upload the jts-1.0.dll, but the ikvm library seems to hit the
file size limit.

However, you should be able to download the ikvm version from
Sourceforge: [link]

Re: [gendarme] Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Thu, 10 Dec 2009 12:54:06 UT

Thanks for confirming!

I think I'll ignore the InvalidOperationException exception when calling
LoadDebuggingSymbols for 2.6 and review this (along with the other
cases) for something better in 2.8.

Sebastien

Re: [gendarme] Re: Gendarme 2.6 Preview 1 is ready for download

j...@nurv.fr (Jb Evain) — Thu, 10 Dec 2009 09:23:41 UT

Hey,

That's indeed a pdb/assembly version mismatch. Which obviously needs a
better description.

--
Jb Evain <j...@nurv.fr>

Re: [gendarme] Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Thu, 10 Dec 2009 01:59:12 UT

Heh, I don't think this is a good example (unless the tool maker wants
to select a few rules to get better generated code).

A lot of the rules (e.g. naming) won't like how Java promotes its API.
Also a lot of code is "compiler" generated (I'll bet without a generated
code attribute) and likely contains quite a bit of code duplication.

Re: [gendarme] Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Thu, 10 Dec 2009 01:50:49 UT

Ah, that makes sense.

I wonder if the exception is about that mismatch ?!? I'll ask JB (cc).
If the exception is right it needs a better description :-)

Sebastien

Re: [gendarme] Re: Gendarme 2.6 Preview 1 is ready for download

sebastien.poul...@gmail.com (Sebastien Pouliot) — Thu, 10 Dec 2009 01:48:19 UT

You should be able to upload it now. I'll keep a copy of it in my
"testcases" directory for use in the future.

Thanks
Sebastien