summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2018-02-10 18:28:52 +0100
committerBram Moolenaar <Bram@vim.org>2018-02-10 18:28:52 +0100
commit42443c7d7fecc3a2a72154bb6139b028438617c2 (patch)
tree049f0045d9299a9d630af0af0a49524897e8b564
parent5a093437199001a0d60d8e18e2b9539b99a7757c (diff)
patch 8.0.1495: having 'pumwidth' default to zero has no meritv8.0.1495
Problem: Having 'pumwidth' default to zero has no merit. Solution: Make the default 15, as the actual default value.
-rw-r--r--src/option.c2
-rw-r--r--src/popupmnu.c39
-rw-r--r--src/version.c2
3 files changed, 18 insertions, 25 deletions
diff --git a/src/option.c b/src/option.c
index 5e215d52d5..e54ba4e2de 100644
--- a/src/option.c
+++ b/src/option.c
@@ -2245,7 +2245,7 @@ static struct vimoption options[] =
#else
(char_u *)NULL, PV_NONE,
#endif
- {(char_u *)0L, (char_u *)0L} SCRIPTID_INIT},
+ {(char_u *)15L, (char_u *)15L} SCRIPTID_INIT},
{"pythonthreedll", NULL, P_STRING|P_EXPAND|P_VI_DEF|P_SECURE,
#if defined(DYNAMIC_PYTHON3)
(char_u *)&p_py3dll, PV_NONE,
diff --git a/src/popupmnu.c b/src/popupmnu.c
index f53649f123..09d9520a63 100644
--- a/src/popupmnu.c
+++ b/src/popupmnu.c
@@ -67,15 +67,6 @@ pum_compute_size(void)
}
/*
- * Return the minimum width of the popup menu.
- */
- static int
-pum_get_width(void)
-{
- return p_pw == 0 ? PUM_DEF_WIDTH : p_pw;
-}
-
-/*
* Show the popup menu with items "array[size]".
* "array" must remain valid until pum_undisplay() is called!
* When possible the leftmost character is aligned with screen column "col".
@@ -102,7 +93,7 @@ pum_display(
do
{
- def_width = pum_get_width();
+ def_width = p_pw;
above_row = 0;
below_row = cmdline_row;
@@ -225,10 +216,10 @@ pum_display(
if (def_width < max_width)
def_width = max_width;
- if (((col < Columns - pum_get_width() || col < Columns - max_width)
+ if (((col < Columns - p_pw || col < Columns - max_width)
#ifdef FEAT_RIGHTLEFT
&& !curwin->w_p_rl)
- || (curwin->w_p_rl && (col > pum_get_width() || col > max_width)
+ || (curwin->w_p_rl && (col > p_pw || col > max_width)
#endif
))
{
@@ -244,17 +235,17 @@ pum_display(
pum_width = Columns - pum_col - pum_scrollbar;
if (pum_width > max_width + pum_kind_width + pum_extra_width + 1
- && pum_width > pum_get_width())
+ && pum_width > p_pw)
{
/* the width is too much, make it narrower */
pum_width = max_width + pum_kind_width + pum_extra_width + 1;
- if (pum_width < pum_get_width())
- pum_width = pum_get_width();
+ if (pum_width < p_pw)
+ pum_width = p_pw;
}
- else if (((col > pum_get_width() || col > max_width)
+ else if (((col > p_pw || col > max_width)
#ifdef FEAT_RIGHTLEFT
&& !curwin->w_p_rl)
- || (curwin->w_p_rl && (col < Columns - pum_get_width()
+ || (curwin->w_p_rl && (col < Columns - p_pw
|| col < Columns - max_width)
#endif
))
@@ -282,9 +273,9 @@ pum_display(
#endif
pum_width = pum_col - pum_scrollbar;
- if (pum_width < pum_get_width())
+ if (pum_width < p_pw)
{
- pum_width = pum_get_width();
+ pum_width = p_pw;
#ifdef FEAT_RIGHTLEFT
if (curwin->w_p_rl)
{
@@ -300,12 +291,12 @@ pum_display(
}
else if (pum_width > max_width + pum_kind_width
+ pum_extra_width + 1
- && pum_width > pum_get_width())
+ && pum_width > p_pw)
{
pum_width = max_width + pum_kind_width
+ pum_extra_width + 1;
- if (pum_width < pum_get_width())
- pum_width = pum_get_width();
+ if (pum_width < p_pw)
+ pum_width = p_pw;
}
}
@@ -323,8 +314,8 @@ pum_display(
}
else
{
- if (max_width > pum_get_width())
- max_width = pum_get_width(); /* truncate */
+ if (max_width > p_pw)
+ max_width = p_pw; /* truncate */
#ifdef FEAT_RIGHTLEFT
if (curwin->w_p_rl)
pum_col = max_width - 1;
diff --git a/src/version.c b/src/version.c
index f5772a8c26..3a354753a2 100644
--- a/src/version.c
+++ b/src/version.c
@@ -772,6 +772,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 1495,
+/**/
1494,
/**/
1493,