directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Enrique Rodriguez <>
Subject Re: [jira] Commented: (DIRLDAP-31) Commons.Cli implemented in Bind
Date Tue, 01 Feb 2005 16:12:05 GMT
Emmanuel Lecharny wrote:
> I will stick to those conventions. I really like those m_ prefix, as  I
> have spent hours catching those kind of "name = name;" errors. Wathever.
> Good style, bad style, at least have a style! I buy your style, I'm very
> close to it ("Writing" solid code, that was a good basis to begin with.
> M$ should have read it ;)
> I'm working with Eclipse, but I could try a switch. 

Trustin and I use Eclipse.


> I'm still in the process to work out a complete common-cli solution for
> Client (almost done). I'm also fighting against svn diff that does not
> skip spaces differences. May I use : svn diff --diff-cmd /usr/bin/diff
> --extensions '-bBw' ?
> Le mardi 01 février 2005 à 16:34 +0100, Alex Karasulu (JIRA) a écrit :
>>     [
>>Alex Karasulu commented on DIRLDAP-31:
>>I need to clarify what our convention is.  Personally I change as I discover cleaner
styles out there.  Right now I think the cleanest style I have ever encountered is Jason Van
Zyl's.  Just look at any recent commits of mine to see it.  
>>To specifically answer your questions...
>>-> Client code uses old old style way back when so no need to follow
>>-> Keep {} on separate lines always
>>-> space pad ()
>>-> if, else, while, for, try, catch all on new lines
>>-> throw away the m_, a_ and l_ conventions ( go with using this. and super.)
>>-> yeah I need to get off my but and get a Jalopy and/or Checkstyle worked out
for this style: Jason's style is really easy to read on my eyes.
>>-> I use IDEA.  We actually have a license for the directory project that was donated
to us.  Let me know if you would like to use this to write directory code.
>>>Commons.Cli implemented in Bind
>>>         Key: DIRLDAP-31
>>>         URL:
>>>     Project: Directory LDAP
>>>        Type: Improvement
>>>  Components: Clients
>>>    Versions: 0.8.0
>>>    Reporter: Emmanuel Lecharny
>>>    Assignee: Alex Karasulu
>>>    Priority: Minor
>>>I've implemented commons.cli in Bind, it works (also used log4j) :
>>>FATAL org.apache.ldap.clients.LdapBind  - Error while parsing command line arguments
: Unrecognized option: -k
>>>INFO org.apache.ldap.clients.LdapBind  - Command line arguments : -k
-p 389 -D cn=Manager,dc=iktek,dc=com -w secret -v 
>>>usage: Bind
>>> -D,--binddn      bind DN
>>> -P,--protocol    procotol version (default: 3)
>>> -h,--host        LDAP server
>>> -n,--noupdate    show what would be done but don't actually update
>>> -p,--port        port on LDAP server
>>> -v,--verbose     run in verbose mode (diagnostics to standard output)
>>> -w,--passwd      bind passwd (for simple authentication)

View raw message