From: Simon Farre <simon.farre.cx@gmail.com>
To: gdb-patches@sourceware.org
Cc: tom@tromey.com, Simon Farre <simon.farre.cx@gmail.com>
Subject: [PATCH] gdb/DAP Few bug fixes & Evaluate Array Watch vars
Date: Tue, 20 Jun 2023 10:15:49 +0200 [thread overview]
Message-ID: <20230620081549.202591-1-simon.farre.cx@gmail.com> (raw)
This patch fixes a few bugs.
First of all, name of VariableReferences must always be of string type.
This patch makes sure that this is the case by formatting the name. If
(when) the name is an integer, this will cause clients to fail or throw
errors.
Fixes a bug in NoOpArrayPrinter that calculated children to be N, but
only ever retrieves N-1 children, which makes Python at some time later
(during fetch_children -> fetch_one_child(N) ) raise an exception (out
of list index) which makes the entire request go bad.
The result[self.result_name] also f-strings the printer.to_string()
value, because this can potentially be a LazyString (which is a Python
object, not a string) and is not serializable by json.dumps.
---
gdb/python/lib/gdb/dap/evaluate.py | 6 +++---
gdb/python/lib/gdb/dap/varref.py | 8 ++++++--
gdb/python/lib/gdb/printing.py | 2 +-
3 files changed, 10 insertions(+), 6 deletions(-)
diff --git a/gdb/python/lib/gdb/dap/evaluate.py b/gdb/python/lib/gdb/dap/evaluate.py
index af7bf43afd0..382ba00877f 100644
--- a/gdb/python/lib/gdb/dap/evaluate.py
+++ b/gdb/python/lib/gdb/dap/evaluate.py
@@ -26,8 +26,8 @@ from .varref import find_variable, VariableReference
class EvaluateResult(VariableReference):
- def __init__(self, value):
- super().__init__(None, value, "result")
+ def __init__(self, expr, value):
+ super().__init__(expr, value, "result")
# Helper function to evaluate an expression in a certain frame.
@@ -39,7 +39,7 @@ def _evaluate(expr, frame_id):
frame.select()
global_context = False
val = gdb.parse_and_eval(expr, global_context=global_context)
- ref = EvaluateResult(val)
+ ref = EvaluateResult(expr, val)
return ref.to_object()
diff --git a/gdb/python/lib/gdb/dap/varref.py b/gdb/python/lib/gdb/dap/varref.py
index 23f18d647c3..9f6068b017e 100644
--- a/gdb/python/lib/gdb/dap/varref.py
+++ b/gdb/python/lib/gdb/dap/varref.py
@@ -66,7 +66,7 @@ class BaseReference:
"variablesReference": self.ref,
}
if self.name is not None:
- result["name"] = self.name
+ result["name"] = f"{self.name}"
return result
def no_children(self):
@@ -152,7 +152,11 @@ class VariableReference(BaseReference):
def to_object(self):
result = super().to_object()
- result[self.result_name] = self.printer.to_string()
+ # If we're a primitive, gdb.Value f-strings just fine
+ # We also must f-string printer.to_string() as it can be a LazyString
+ result[self.result_name] = (
+ f"{self.value}" if self.ref == 0 else f"{self.printer.to_string()}"
+ )
num_children = self.child_count()
if num_children is not None:
if (
diff --git a/gdb/python/lib/gdb/printing.py b/gdb/python/lib/gdb/printing.py
index ab62ef3e45c..353427d000a 100644
--- a/gdb/python/lib/gdb/printing.py
+++ b/gdb/python/lib/gdb/printing.py
@@ -298,7 +298,7 @@ class NoOpArrayPrinter:
return "array"
def children(self):
- for i in range(self.low, self.high):
+ for i in range(self.low, self.high + 1):
yield (i, self.value[i])
--
2.40.1
next reply other threads:[~2023-06-20 8:16 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-20 8:15 Simon Farre [this message]
2023-06-20 8:25 ` Simon Farre
2023-06-20 14:49 ` Tom Tromey
2023-06-20 18:41 ` Simon Farre
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230620081549.202591-1-simon.farre.cx@gmail.com \
--to=simon.farre.cx@gmail.com \
--cc=gdb-patches@sourceware.org \
--cc=tom@tromey.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).