From 8210693795d6d0d51bf5b70674d4539cdde0330b Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Fri, 13 Mar 2020 14:34:38 +0100 Subject: 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. --- src/popupwin.c | 48 +++++++++++++++++++++++++++--------------------- 1 file changed, 27 insertions(+), 21 deletions(-) (limited to 'src/popupwin.c') diff --git a/src/popupwin.c b/src/popupwin.c index 1b56c1c4b..8e981b0be 100644 --- 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. -- cgit v1.2.1