patch 9.1.1109: cmdexpand.c hard to read

Problem:  cmdexpand.c hard to read
Solution: refactor the file slightly (glepnir)

closes: #16621

Signed-off-by: glepnir <glephunter@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>

src/cmdexpand.c

@@ -373,10 +373,7 @@ cmdline_pum_create(
 	columns += vim_strsize(showmatches_gettail(matches[0]));
 	columns -= vim_strsize(matches[0]);
     }
-    if (columns >= compl_startcol)
+    compl_startcol = MAX(0, compl_startcol - columns);
-	compl_startcol = 0;
-    else
-	compl_startcol -= columns;
 
     // no default selection
     compl_selected = -1;
@@ -735,25 +732,35 @@ win_redr_status_matches(
  * in "xp->xp_selected"
  */
     static char_u *
-get_next_or_prev_match(
+get_next_or_prev_match(int mode, expand_T *xp)
-	int		mode,
-	expand_T	*xp)
 {
     int	    findex = xp->xp_selected;
     int	    ht;
 
+    // When no files found, return NULL
     if (xp->xp_numfiles <= 0)
 	return NULL;
 
     if (mode == WILD_PREV)
     {
+	// Select last file if at start
 	if (findex == -1)
 	    findex = xp->xp_numfiles;
 	--findex;
     }
     else if (mode == WILD_NEXT)
-	++findex;
+    {
-    else if (mode == WILD_PAGEUP)
+	// Select next file
+	findex = findex + 1;
+    }
+    else   // WILD_PAGEDOWN or WILD_PAGEUP
+    {
+	// Get the height of popup menu (used for both PAGEUP and PAGEDOWN)
+	ht = pum_get_height();
+	if (ht > 3)
+	    ht -= 2;
+
+	if (mode == WILD_PAGEUP)
 	{
 	    if (findex == 0)
 		// at the first entry, don't select any entries
@@ -762,67 +769,47 @@ get_next_or_prev_match(
 		// no entry is selected. select the last entry
 		findex = xp->xp_numfiles - 1;
 	    else
-	{
 		// go up by the pum height
-	    ht = pum_get_height();
+		findex = MAX(findex - ht, 0);
-	    if (ht > 3)
-		ht -= 2;
-	    findex -= ht;
-	    if (findex < 0)
-		// few entries left, select the first entry
-		findex = 0;
-	}
 	}
 	else    // mode == WILD_PAGEDOWN
 	{
-	if (findex == xp->xp_numfiles - 1)
+	    if (findex < 0)
+		// no entry is selected, select the first entry
+		findex = 0;
+	    else if (findex >= xp->xp_numfiles - 1)
 		// at the last entry, don't select any entries
 		findex = -1;
-	else if (findex == -1)
-	    // no entry is selected. select the first entry
-	    findex = 0;
 	    else
-	{
 		// go down by the pum height
-	    ht = pum_get_height();
+		findex = MIN(findex + ht, xp->xp_numfiles - 1);
-	    if (ht > 3)
-		ht -= 2;
-	    findex += ht;
-	    if (findex >= xp->xp_numfiles)
-		// few entries left, select the last entry
-		findex = xp->xp_numfiles - 1;
 	}
     }
 
-    // When wrapping around, return the original string, set findex to -1.
+    // Handle wrapping around
-    if (findex < 0)
+    if (findex < 0 || findex >= xp->xp_numfiles)
     {
-	if (xp->xp_orig == NULL)
+	// If original string exists, return to it when wrapping around
-	    findex = xp->xp_numfiles - 1;
+	if (xp->xp_orig != NULL)
-	else
 	    findex = -1;
-    }
-    if (findex >= xp->xp_numfiles)
-    {
-	if (xp->xp_orig == NULL)
-	    findex = 0;
 	else
-	    findex = -1;
+	    // Wrap around to opposite end
+	    findex = (findex < 0) ? xp->xp_numfiles - 1 : 0;
     }
+
+    // Display matches on screen
     if (compl_match_array)
     {
 	compl_selected = findex;
 	cmdline_pum_display();
     }
     else if (p_wmnu)
-	win_redr_status_matches(xp, xp->xp_numfiles, xp->xp_files,
+	win_redr_status_matches(xp, xp->xp_numfiles, xp->xp_files, findex,
-		findex, cmd_showtail);
+		cmd_showtail);
+
     xp->xp_selected = findex;
-
+    // Return the original string or the selected match
-    if (findex == -1)
+    return vim_strsave(findex == -1 ? xp->xp_orig : xp->xp_files[findex]);
-	return vim_strsave(xp->xp_orig);
-
-    return vim_strsave(xp->xp_files[findex]);
 }
 
 /*

src/version.c

@@ -704,6 +704,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1109,
 /**/
     1108,
 /**/