directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Karasulu <aok...@bellsouth.net>
Subject Re: [jira] Commented: (DIRLDAP-31) Commons.Cli implemented in Bind
Date Tue, 01 Feb 2005 16:33: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 ;)
>  
>
Yeah I used to be big on m_ too.  I started the m_ thing and went 
overboard with a_ and l_.  I was out of control.  Anyway I'm realizing 
good ways to do things from different communities here at ASF, and 
others outside.  I like reading peoples code and certain styles are 
easier to read.

>I'm working with Eclipse, but I could try a switch. 
>  
>
Enrique may prefer you stick to Eclipse hehe.

>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' ?
>  
>
Use what what you like so long as the code compiles and works.  Just let 
me know what to use to apply the patch if it's non-standard.  

Good job groking the code.

Thanks,
Alex

>
>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