neovim

commit a217675a67233ca2032cd668e919858d2aed92e7
parent fcaf0d5f3824431520a0b6353c771107cc63ed4e
Author: Famiu Haque <famiuhaque@proton.me>
Date:   Wed,  7 Jun 2023 08:51:24 +0600

refactor(options): use slash separator for `option_get_valid_types()` (#23945)

`option_get_valid_types()` currently uses a comma separator for
multi-type options which does not fit well with the changed error
message for invalid option value type. A slash seperator is much more
suited for its current use-case.
Diffstat:
M
src/nvim/option.c
 | 
6
+++---

1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/nvim/option.c b/src/nvim/option.c
@@ -3428,8 +3428,8 @@ static char *optval_to_cstr(OptVal o)
 
 // Get an allocated string containing a list of valid types for an option.
 // For options with a singular type, it returns the name of the type. For options with multiple
-// possible types, it returns a comma separated list of types. For example, if an option can be a
-// number, boolean or string, the function returns "Number, Boolean, String"
+// possible types, it returns a slash separated list of types. For example, if an option can be a
+// number, boolean or string, the function returns "Number/Boolean/String"
 static char *option_get_valid_types(int opt_idx)
 {
   uint32_t flags = options[opt_idx].flags;
@@ -3443,7 +3443,7 @@ static char *option_get_valid_types(int opt_idx)
     if (type_count == 0) { \
       kv_concat(str, typename); \
     } else { \
-      kv_printf(str, ", %s", typename); \
+      kv_printf(str, "/%s", typename); \
     } \
     type_count++; \
   } while (0);