patch 8.2.0375: Coverity warning for not using return value

Problem:    Coverity warning for not using return value.
Solution:   Move error message to separate function.

diff --git a/src/popupwin.c b/src/popupwin.c
index 1b56c1c4bede7f846502c0e632c7a044083c8e65..8e981b0beccf1a27c86674eedb3eaf0e6c01af2a 100644 (file)
--- a/src/popupwin.c
+++ b/src/popupwin.c
@@ -2492,6 +2492,31 @@ popup_free(win_T *wp)
     popup_mask_refresh = TRUE;
 }
 
+    static void
+error_for_popup_window(void)
+{
+    emsg(_("E994: Not allowed in a popup window"));
+}
+
+    int
+error_if_popup_window(int also_with_term UNUSED)
+{
+    // win_execute() may set "curwin" to a popup window temporarily, but many
+    // commands are disallowed then.  When a terminal runs in the popup most
+    // things are allowed.  When a terminal is finished it can be closed.
+    if (WIN_IS_POPUP(curwin)
+# ifdef FEAT_TERMINAL
+           && (also_with_term || curbuf->b_term == NULL)
+           && !term_is_finished(curbuf)
+# endif
+           )
+    {
+       error_for_popup_window();
+       return TRUE;
+    }
+    return FALSE;
+}
+
 /*
  * Close a popup window by Window-id.
  * Does not invoke the callback.
@@ -2509,7 +2534,7 @@ popup_close(int id)
        {
            if (wp == curwin)
            {
-               ERROR_IF_ANY_POPUP_WINDOW;
+               error_for_popup_window();
                return;
            }
            if (prev == NULL)
@@ -2540,7 +2565,7 @@ popup_close_tabpage(tabpage_T *tp, int id)
        {
            if (wp == curwin)
            {
-               ERROR_IF_ANY_POPUP_WINDOW;
+               error_for_popup_window();
                return;
            }
            if (prev == NULL)
@@ -2888,25 +2913,6 @@ f_popup_getoptions(typval_T *argvars, typval_T *rettv)
     }
 }
 
-    int
-error_if_popup_window(int also_with_term UNUSED)
-{
-    // win_execute() may set "curwin" to a popup window temporarily, but many
-    // commands are disallowed then.  When a terminal runs in the popup most
-    // things are allowed.  When a terminal is finished it can be closed.
-    if (WIN_IS_POPUP(curwin)
-# ifdef FEAT_TERMINAL
-           && (also_with_term || curbuf->b_term == NULL)
-           && !term_is_finished(curbuf)
-# endif
-           )
-    {
-       emsg(_("E994: Not allowed in a popup window"));
-       return TRUE;
-    }
-    return FALSE;
-}
-
 # if defined(FEAT_TERMINAL) || defined(PROTO)
 /*
  * Return TRUE if the current window is running a terminal in a popup window.

diff --git a/src/version.c b/src/version.c
index 6f1b9e79364e79addff43f538e8f33dc2244e885..2e720adce445c4ad3320f79aba001e9b4192bc30 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -738,6 +738,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    375,
 /**/
     374,
 /**/