Re: [PATCH v2 05/10] nfsdcltrack: add a new "one-shot" program for manipulating the client tracking db

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 25/10/12 09:53, Jeff Layton wrote:
>> Is needs_arg really necessary? Once the function is found, just pass 
>> > it the rest of argv and let the function decide if there should
>> > be an argument or not... but again.. just a nit... 
>> > 
>> > steved. 
> No, it's not strictly necessary. The alternative though is to make the
> command functions do argv processing, which seemed more ugly to me. I
> tend to prefer to keep argv processing in a single place since it can
> get sort of hairy to work through the bounds checks after getopt and
> such.
> 
> If you feel strongly about it, I can try to change it, but I don't
> think the result will be cleaner.
> 
Like I said its a nit... Just a different way to look at it... no
issues here keeping it as is... 

steved.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux