On Wed, Jan 28, 2009 at 1:40 PM, Perry Trolard <trol...@gmail.com> wrote:
>
> Is there anything I can do to help get the proposed change above, or
> something like it, included in c.c.command-line? I'd be happy to
> modify the patch: removing the somewhat orthogonal alignment code,
> etc.

I've been conflicted over how to proceed with this lib -- thanks for
your patience.  With a few small changes I'd be happy to include this.

Please make the 'justify' fn (whatever you want to call it) private
for now so that it doesn't become another API to be maintained.  If it
proves to be more generally useful, it can be moved to a more
approriate lib later.

Printing the options as --(a|b) is rather unusual and likely to be
confusing.  More common appears to be like this line from "ls --help":

  -r, --reverse              reverse order while sorting

Please use something like this instead.  Showing double-dashes for
each option is probably fine for now.  If we want to allow combining
of multiple single-letter options without spaces between them later on
(which itself has questionable value) -- well, that's a question for
another time.

Finally, please don't include any commented-out lines in your final
patch.

Thanks so much for your help,
--Chouser

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To post to this group, send email to clojure@googlegroups.com
To unsubscribe from this group, send email to 
clojure+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to