From patchwork Thu Aug 1 22:34:20 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Terekhov, Mikhail via Gdb-patches" X-Patchwork-Id: 33913 Received: (qmail 39728 invoked by alias); 1 Aug 2019 22:34:27 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 39716 invoked by uid 89); 1 Aug 2019 22:34:27 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-24.2 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.1 spammy=479, extends, H*MI:google X-HELO: mail-yw1-f73.google.com Received: from mail-yw1-f73.google.com (HELO mail-yw1-f73.google.com) (209.85.161.73) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 01 Aug 2019 22:34:26 +0000 Received: by mail-yw1-f73.google.com with SMTP id k21so54337105ywk.2 for ; Thu, 01 Aug 2019 15:34:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:message-id:mime-version:subject:from:to:cc; bh=4Je7Dr5f8HT4vD/+XO6OaA/5IecTUX6lbbiIxsgIV/Q=; b=CKf6B445slmSmT5lsdCSECANBrazwNVx8B6a7zvMVqeYgxvrp+/O34Yzq4MyQjwlCs sExbDi+017dsErOsDj5QcD2c2mbxEhgW1ZuvHmGPOBWkIVtyKwh+x+70sfPcDed8aWlD DUHjJTV52gKVe9cQTJ4iHC87h4JHvzKbOmOSGDCYTwX4iLSR7zsL20+HcUklAu53otdb D5gJr0fbL8Y+Llqd8QsRw7qqIYeTe6bH2GE6Eg+uTpk5ptR/CBFSYEVOnOSy8YgCe13c Tw1/NmYgwuA/YMwlCRIxjNQIKqlMb729Pdr2vmO3SWYtx64sMyhHMVOqg/HzByH0eYEn obpw== Date: Thu, 1 Aug 2019 17:34:20 -0500 Message-Id: <20190801223420.234581-1-cbiesinger@google.com> Mime-Version: 1.0 Subject: [PATCH] Add block['var'] accessor X-Patchwork-Original-From: "Christian Biesinger via gdb-patches" From: "Terekhov, Mikhail via Gdb-patches" Reply-To: Christian Biesinger To: gdb-patches@sourceware.org Cc: Christian Biesinger Currently we support iteration on blocks; this patch extends that to make subscript access work as well. gdb/ChangeLog: 2019-08-01 Christian Biesinger * NEWS: Mention dictionary access on blocks. * python/py-block.c (blpy_getitem): New function. (block_object_as_mapping): New struct. (block_object_type): Use new struct for tp_as_mapping field. gdb/doc/ChangeLog: 2019-08-01 Christian Biesinger * python.texi (Blocks In Python): Document dictionary access on blocks. gdb/testsuite/ChangeLog: 2019-08-01 Christian Biesinger * gdb.python/py-block.exp: Test dictionary access on blocks. --- gdb/NEWS | 3 +++ gdb/doc/python.texi | 7 +++++- gdb/python/py-block.c | 32 ++++++++++++++++++++++++++- gdb/testsuite/gdb.python/py-block.exp | 5 +++++ 4 files changed, 45 insertions(+), 2 deletions(-) diff --git a/gdb/NEWS b/gdb/NEWS index 3c3351e2c4..2db86c2ae7 100644 --- a/gdb/NEWS +++ b/gdb/NEWS @@ -47,6 +47,9 @@ ** gdb.Objfile has new methods 'lookup_global_symbol' and 'lookup_static_symbol' to lookup a symbol from this objfile only. + ** gdb.Block now supports the dictionary syntax for accessing symbols in + this block (e.g. block['local_variable']). + * New commands | [COMMAND] | SHELL_COMMAND diff --git a/gdb/doc/python.texi b/gdb/doc/python.texi index 3fdccd5e43..832283dede 100644 --- a/gdb/doc/python.texi +++ b/gdb/doc/python.texi @@ -4722,7 +4722,12 @@ A @code{gdb.Block} is iterable. The iterator returns the symbols should not assume that a specific block object will always contain a given symbol, since changes in @value{GDBN} features and infrastructure may cause symbols move across blocks in a symbol -table. +table. You can also use Python's @dfn{dictionary syntax} to access +variables in this block, e.g.: + +@smallexample +symbol = some_block['variable'] # symbol is of type gdb.Symbol +@end smallexample The following block-related functions are available in the @code{gdb} module: diff --git a/gdb/python/py-block.c b/gdb/python/py-block.c index 90140ebc34..94d2454533 100644 --- a/gdb/python/py-block.c +++ b/gdb/python/py-block.c @@ -224,6 +224,30 @@ blpy_is_static (PyObject *self, void *closure) Py_RETURN_FALSE; } +/* Given a string, returns the gdb.Symbol representing that symbol in this + block. If such a symbol does not exist, returns NULL with a Python + exception. */ + +static PyObject * +blpy_getitem (PyObject *self, PyObject *key) +{ + const struct block *block; + + BLPY_REQUIRE_VALID (self, block); + + gdb::unique_xmalloc_ptr name = python_string_to_host_string (key); + if (name == nullptr) + return nullptr; + + struct symbol *sym = block_lookup_symbol ( + block, name.get(), symbol_name_match_type::FULL, VAR_DOMAIN); + if (!sym) { + PyErr_SetObject (PyExc_KeyError, key); + return nullptr; + } + return symbol_to_symbol_object (sym); +} + static void blpy_dealloc (PyObject *obj) { @@ -440,6 +464,12 @@ static gdb_PyGetSetDef block_object_getset[] = { { NULL } /* Sentinel */ }; +static PyMappingMethods block_object_as_mapping = { + NULL, + blpy_getitem, + NULL +}; + PyTypeObject block_object_type = { PyVarObject_HEAD_INIT (NULL, 0) "gdb.Block", /*tp_name*/ @@ -453,7 +483,7 @@ PyTypeObject block_object_type = { 0, /*tp_repr*/ 0, /*tp_as_number*/ 0, /*tp_as_sequence*/ - 0, /*tp_as_mapping*/ + &block_object_as_mapping, /*tp_as_mapping*/ 0, /*tp_hash */ 0, /*tp_call*/ 0, /*tp_str*/ diff --git a/gdb/testsuite/gdb.python/py-block.exp b/gdb/testsuite/gdb.python/py-block.exp index 20d3968c04..6be1abe7ae 100644 --- a/gdb/testsuite/gdb.python/py-block.exp +++ b/gdb/testsuite/gdb.python/py-block.exp @@ -43,6 +43,11 @@ gdb_test "python print (block)" "" "check block not No gdb_test "python print (block.function)" "None" "first anonymous block" gdb_test "python print (block.start)" "${decimal}" "check start not None" gdb_test "python print (block.end)" "${decimal}" "check end not None" +gdb_test "python print (block\['f'\].name == 'f')" "True" "check variable access" +gdb_test "python print (block\['nonexistent'\])" ".*KeyError: 'nonexistent'.*" \ + "check nonexistent variable" +gdb_test "python print (block\[42\])" ".*TypeError: Expected a string.*" \ + "check non-string key" # Test global/static blocks gdb_py_test_silent_cmd "python frame = gdb.selected_frame()" "Get Frame" 0