Re: [Patch 6/10] NFS Mount Configuration File (Vers 3)

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

 



On 08/06/2009 04:44 PM, Chuck Lever wrote:
> 
> Seems to me the text-based mount option parser routines (parse_opt.[ch])
> already does a lot of this.  It can break up a string into a linked list
> of options, handle quoted commas properly, catenate options to the list,
> and flatten the list back into a C string.
Well I patterned the code after the file parsing code in idmap daemon
which means there a common theme between both the idmap configuration
file and the mount configuration. I think that is important. 
 
> 
> Wouldn't it be easier to adapt that code rather than adding new code
> that does nearly the same thing?  It would save you a lot of C string
> manipulation, and reuse working code.
At this point, no. and I am reuse a lot of code...

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