From 190d0241e2a0cd4ec4bcb520772db835df935530 Mon Sep 17 00:00:00 2001
From: zeertzjq <zeertzjq@outlook.com>
Date: Wed, 19 Mar 2025 14:17:53 +0800
Subject: fix(options): fix 'winborder' accepting multiple string values
 (#32978)

Problem:  'winborder' accepting multiple string values.
Solution: Use the fallback did_set_str_generic() callback instead of
          did_set_winborder() which calls opt_strings_flags() with
          incorrect last argument.
---
 src/nvim/options.lua | 1 -
 src/nvim/optionstr.c | 9 ---------
 2 files changed, 10 deletions(-)

(limited to 'src')

diff --git a/src/nvim/options.lua b/src/nvim/options.lua
index 3cf60513c4..6c71477ba0 100644
--- a/src/nvim/options.lua
+++ b/src/nvim/options.lua
@@ -10190,7 +10190,6 @@ local options = {
     },
     {
       defaults = { if_true = '' },
-      cb = 'did_set_winborder',
       values = { '', 'double', 'single', 'shadow', 'rounded', 'solid', 'none' },
       desc = [=[
         Defines the default border style of floating windows. The default value
diff --git a/src/nvim/optionstr.c b/src/nvim/optionstr.c
index c3cdd6e804..3a6b4c9936 100644
--- a/src/nvim/optionstr.c
+++ b/src/nvim/optionstr.c
@@ -2005,15 +2005,6 @@ const char *did_set_winhighlight(optset_T *args)
   return NULL;
 }
 
-/// The 'winborder' option is changed.
-const char *did_set_winborder(optset_T *args)
-{
-  if (opt_strings_flags(p_winbd, opt_winborder_values, NULL, true) != OK) {
-    return e_invarg;
-  }
-  return NULL;
-}
-
 int expand_set_winhighlight(optexpand_T *args, int *numMatches, char ***matches)
 {
   return expand_set_opt_generic(args, get_highlight_name, numMatches, matches);
-- 
cgit