From: Chuck Lever Date: Thu, 11 Oct 2007 17:55:02 +0000 (-0400) Subject: text-based mount.nfs: Fix po_rightmost() enum return values X-Git-Tag: nfs-utils-1-1-1~10 X-Git-Url: https://git.decadent.org.uk/gitweb/?p=nfs-utils.git;a=commitdiff_plain;h=da7454e3cf772edacc55a0f47d49cef7a03a4d10 text-based mount.nfs: Fix po_rightmost() enum return values Neil observed that po_rightmost() now returns enum values from both enum { PO_NOT_FOUND = 0, PO_FOUND = 1, } and enum { PO_KEY2_RIGHTMOST = 1, PO_KEY1_RIGHTMOST = -1, } It would be cleaner to use a single enum for po_rightmost()'s return value. We take the next logical step and create specific types for the return values in order to ensure we don't mix the enum values, and to document explicitly what return values callers can expect. This could have been a simpler patch, but I think the end result is a cleaner overall parser API. Signed-off-by: Chuck Lever Signed-off-by: Neil Brown --- diff --git a/utils/mount/parse_opt.c b/utils/mount/parse_opt.c index f253142..cb398bd 100644 --- a/utils/mount/parse_opt.c +++ b/utils/mount/parse_opt.c @@ -255,13 +255,12 @@ void po_replace(struct mount_options *target, struct mount_options *source) * Convert our mount options object back into a string that the * rest of the world can use. * - * Returns 1 if the string was successfully created; otherwise - * zero. Upon return, @string contains the address of a - * replacement C string containing a comma-delimited list of - * mount options and values; or the passed-in string is freed - * and NULL is returned if some failure occurred. + * Upon return, @string contains the address of a replacement + * C string containing a comma-delimited list of mount options + * and values; or the passed-in string is freed and NULL is + * returned if some failure occurred. */ -int po_join(struct mount_options *options, char **str) +po_return_t po_join(struct mount_options *options, char **str) { size_t len = 0; struct mount_option *option; @@ -310,10 +309,8 @@ int po_join(struct mount_options *options, char **str) * @options: pointer to mount options * @option: pointer to a C string containing the option to add * - * Returns 1 if the list was successfully concatenated; otherwise - * zero. */ -int po_append(struct mount_options *options, char *str) +po_return_t po_append(struct mount_options *options, char *str) { struct mount_option *option = option_create(str); @@ -329,9 +326,8 @@ int po_append(struct mount_options *options, char *str) * @options: pointer to mount options * @keyword: pointer to a C string containing option keyword for which to search * - * Returns 1 if the option is present in the list; otherwise zero. */ -int po_contains(struct mount_options *options, char *keyword) +po_found_t po_contains(struct mount_options *options, char *keyword) { struct mount_option *option; @@ -382,12 +378,9 @@ char *po_get(struct mount_options *options, char *keyword) * * This function can be used to determine which of two similar * options will be the one to take effect. - * - * Returns 1 if key2 is rightmost or key1 is not present. - * Returns -1 if key1 is rightmost or key2 is not present. - * Returns 0 if neither key is present. */ -int po_rightmost(struct mount_options *options, char *key1, char *key2) +po_rightmost_t po_rightmost(struct mount_options *options, + char *key1, char *key2) { struct mount_option *option; @@ -400,7 +393,7 @@ int po_rightmost(struct mount_options *options, char *key1, char *key2) } } - return PO_NOT_FOUND; + return PO_NEITHER_FOUND; } /** @@ -408,10 +401,9 @@ int po_rightmost(struct mount_options *options, char *key1, char *key2) * @options: pointer to mount options * @keyword: pointer to a C string containing an option keyword to remove * - * Returns 1 if the option was found and removed; passed-in list is - * truncated upon return; otherwise zero. + * Side-effect: the passed-in list is truncated on success. */ -int po_remove_all(struct mount_options *options, char *keyword) +po_found_t po_remove_all(struct mount_options *options, char *keyword) { struct mount_option *option, *next; int found = PO_NOT_FOUND; diff --git a/utils/mount/parse_opt.h b/utils/mount/parse_opt.h index 6b9fb5e..e7924dd 100644 --- a/utils/mount/parse_opt.h +++ b/utils/mount/parse_opt.h @@ -21,31 +21,32 @@ * */ -enum { +typedef enum { PO_FAILED = 0, PO_SUCCEEDED = 1, -}; +} po_return_t; -enum { +typedef enum { PO_NOT_FOUND = 0, PO_FOUND = 1, -}; +} po_found_t; -enum { - PO_KEY2_RIGHTMOST = 1, +typedef enum { PO_KEY1_RIGHTMOST = -1, -}; + PO_NEITHER_FOUND = 0, + PO_KEY2_RIGHTMOST = 1, +} po_rightmost_t; struct mount_options; struct mount_options * po_split(char *); void po_replace(struct mount_options *, struct mount_options *); -int po_join(struct mount_options *, char **); +po_return_t po_join(struct mount_options *, char **); -int po_append(struct mount_options *, char *); -int po_contains(struct mount_options *, char *); +po_return_t po_append(struct mount_options *, char *); +po_found_t po_contains(struct mount_options *, char *); char * po_get(struct mount_options *, char *); -int po_rightmost(struct mount_options *, char *, char *); -int po_remove_all(struct mount_options *, char *); +po_rightmost_t po_rightmost(struct mount_options *, char *, char *); +po_found_t po_remove_all(struct mount_options *, char *); void po_destroy(struct mount_options *);