directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lecharny <elecha...@iktek.com>
Subject Re: [jira] Commented: (DIRLDAP-31) Commons.Cli implemented in Bind
Date Tue, 01 Feb 2005 16:09:49 GMT
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. 

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 :
>      [ http://issues.apache.org/jira/browse/DIRLDAP-31?page=comments#action_58406 ]
>      
> 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.
> 
> Cheers,
> -Alex
> 
> > Commons.Cli implemented in Bind
> > -------------------------------
> >
> >          Key: DIRLDAP-31
> >          URL: http://issues.apache.org/jira/browse/DIRLDAP-31
> >      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 127.0.0.1 -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)
> 



Mime
View raw message