[sheepdog] [PATCH 1/2] collie: add self options to collie's command
Liu Yuan
namei.unix at gmail.com
Mon Aug 20 18:19:17 CEST 2012
On 08/20/2012 10:28 PM, Yunkai Zhang wrote:
> Now, all collie's command share the same global collie_options, it will
> lead to option's name conflict among commands if they use the same options
> but with different description.
>
> By introducing self options to each command (if necessary) and make
> collie_options only contain the common part of all options, we can solve this
> issue.
I like this improvement, but 'self options' doesn't explain the idea better. This is
kind of namespace for each sub command, so simply name it as:
+ struct sd_option *options;
in a structure is enough.
And rework the comment and commit log to replace 'self option' with more meaningful
phrase such as, for e.g, "By moving the global options into individual structure as a
private member, we can solve this problem"
Also, with this patch, we can then change all those upper cased options into lower cases,
such as vdi create -P -> vdi create -p for easier typing.
--
thanks,
Yuan
More information about the sheepdog
mailing list