[PATCH 2/4] qh: Fall for peer pressure from myself, and implement some help
Andreas Ericsson
ae at op5.se
Thu Nov 29 11:40:51 CET 2012
On 11/28/2012 03:37 PM, robin.sonefors at op5.com wrote:
>> First off, the nerd channel list must be generated dynamically. We
>> have them in a handy array, so we might as well print them all
>> while we're at it. If we make it possible to provide a description
>> string when creating the channel we can print that too and the help
>> API for the nerd channel is pretty much complete.
>
> Right. Sure. I guess I was just wary of API modifications, but no problem to do
> so. Split that into it's own patch.
>
Well, 4.0 isn't released yet, so minor mods are ok.
>> Why not just 'help'? That way people can do what's natural
>> with text interfaces and try "@nerd help" and they would still get the
>> same response, whereas typing "@nerd --help" into an interactive text
>> listener seems less intuitive.
>
> My reasoning was to have something obscure enough that it would be unlikely to
> clash with other commands. I realize now that was pretty crappy reasoning -
> what else would be called "help"?
>
> Note that this feedback affects the first patch in my last patch series, so
> I've changed that patch, rather than change it in a new patch on top of that
> one.
>
> In other words, these three patches replace the first two patches in the old
> series, but the last two in the last series still applies. I'll do things
> differently if you tell me how.
>
I've revamped them a bit myself. I should almost certainly have told you,
since you sit about three feet away from me, but I forgot. Mea culpa. The
nerd_channel->description stuff is something I haven't gotten round to
yet though, so I'll take those. Hold off on new stuff in this area until
I've shipped the modified patches upstream.
--
Andreas Ericsson andreas.ericsson at op5.se
OP5 AB www.op5.se
Tel: +46 8-230225 Fax: +46 8-230231
Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.
------------------------------------------------------------------------------
Keep yourself connected to Go Parallel:
VERIFY Test and improve your parallel project with help from experts
and peers. http://goparallel.sourceforge.net
More information about the Developers
mailing list