[Mono-list] .NET C# LDAP Libraray.

Ben Maurer bmaurer@users.sourceforge.net
Thu, 13 Nov 2003 19:17:36 -0500


Hello Matt,

I was taking a quick look at your sources, they look great! One thing I
did notice, however, is that you guys are naming your classes things
like:
	LDAPFoobar

However, in c# the guidelines for when you have a situation like that:

1) if the first part is two letters, keep it uppercase (eg, IPAddress)
2) otherwise, only have the first letter uppercase (XmlTextWriter,
LdapFoobar)

Right now, you are early enough in the design process that this can
easily be fixed with a tad of grep magic (no need to worry about
history-- there is nothing to loose).

I would highly advise running FxCop (i realize it only runs in Windows,
some day I think we should make something like it for mono).

Better to catch stuff like this early when you dont have to take compat,
cvs history etc into account.

-- Ben