bug-gnu-emacs
[Top][All Lists]
Advanced

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

bug#23720: 25.0.94; Issues with GUD (gdb-mi) after upgrade from Emacs 23


From: Guilhem Bichot
Subject: bug#23720: 25.0.94; Issues with GUD (gdb-mi) after upgrade from Emacs 23 to 24/25
Date: Thu, 9 Jun 2016 09:42:23 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0

Hello,

Eli Zaretskii a écrit le 07/06/2016 20:54 :
Date: Tue, 07 Jun 2016 20:15:34 +0300
From: Eli Zaretskii <eliz@gnu.org>
Cc: 23720@debbugs.gnu.org

ISSUE 3: STEPPING OUT DOESN'T PRINT RETURN VALUE
================================================

The program is in the stepped-in function, clicking "Step out" steps out
of it, but this doesn't print the returned value.
Emacs23 prints it ("Value returned is $1 = false").

Looks like a missing feature in gdb-mi: the return value sent by MI is
not processed.

Please try the patch below, I think it will fix this.

diff --git a/lisp/progmodes/gdb-mi.el b/lisp/progmodes/gdb-mi.el
index 5ad101d..195acaf 100644
--- a/lisp/progmodes/gdb-mi.el
+++ b/lisp/progmodes/gdb-mi.el
@@ -2488,7 +2488,9 @@ gdb-stopped
    ;; Reason is available with target-async only
    (let* ((result (gdb-json-string output-field))
           (reason (bindat-get-field result 'reason))
-         (thread-id (bindat-get-field result 'thread-id)))
+         (thread-id (bindat-get-field result 'thread-id))
+         (retval (bindat-get-field result 'return-value))
+         (varnum (bindat-get-field result 'gdb-result-var)))

      ;; -data-list-register-names needs to be issued for any stopped
      ;; thread
@@ -2514,6 +2516,11 @@ gdb-stopped
      (if (string-equal reason "exited-normally")
        (setq gdb-active-process nil))

+    (when (and retval varnum)
+      (setq gdb-filter-output
+            (concat gdb-filter-output
+                    (format "Value returned is %s = %s\n" varnum retval))))
+
      ;; Select new current thread.

      ;; Don't switch if we have no reasons selected

yes, it works! thanks!
Please, could you consider incorporating this into the next releases?





reply via email to

[Prev in Thread] Current Thread [Next in Thread]