[vte] widget: Only set out param on success



commit d4d101d97f0592bf0a1b9b89e4b1cf2c3dc57f9e
Author: Christian Persch <chpe src gnome org>
Date:   Wed Apr 29 00:24:54 2020 +0200

    widget: Only set out param on success

 src/vtegtk.cc | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)
---
diff --git a/src/vtegtk.cc b/src/vtegtk.cc
index 5c1776b3..4c0fd1b0 100644
--- a/src/vtegtk.cc
+++ b/src/vtegtk.cc
@@ -2293,16 +2293,17 @@ vte_terminal_event_check_regex_array(VteTerminal *terminal,
                                      gsize *n_matches)
 {
         auto matches = vte::glib::take_free_ptr(g_new0(char*, n_regexes));
+        if (!vte_terminal_event_check_regex_simple(terminal,
+                                                   event,
+                                                   regexes,
+                                                   n_regexes,
+                                                   match_flags,
+                                                   matches.get()))
+            return nullptr;
+
         if (n_matches)
                 *n_matches = n_regexes;
-
-        auto const rv = vte_terminal_event_check_regex_simple(terminal,
-                                                              event,
-                                                              regexes,
-                                                              n_regexes,
-                                                              match_flags,
-                                                              matches.get());
-        return rv ? matches.release() : nullptr;
+        return matches.release();
 }
 
 /**


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]