From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1256) id 4D41F3856DD2; Fri, 21 Oct 2022 07:56:26 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4D41F3856DD2 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1666338986; bh=7U+3K/6AEVashgkoHOmrZ5N2WII6LoPJJqw5KY8wmZk=; h=From:To:Subject:Date:From; b=hfimLS7Q0dUgm8MPotiAnNMZy++mZ8KxBLrUrdJPWcT+koXPstrNxDlNVFfQXNmdd sBiYeIDVKVtGJ1QrIOUCgKLroJMvcl9390TWzemkEiNsvWvrSwYuEQv8ea1x/9C+y7 8/iaYTyN/nuIJxds5wlRuPt6CXczl9HjktiWY1KI= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Maciej W. Rozycki To: gdb-cvs@sourceware.org Subject: [binutils-gdb] GDB/Python: Make `None' stand for `unlimited' in setting integer parameters X-Act-Checkin: binutils-gdb X-Git-Author: Maciej W. Rozycki X-Git-Refname: refs/heads/master X-Git-Oldrev: e7e1f2034567207e5e01cb75ea2ffd568a64e84d X-Git-Newrev: c506be7d9be55721a44d38782309d8f7fcd5e99e Message-Id: <20221021075626.4D41F3856DD2@sourceware.org> Date: Fri, 21 Oct 2022 07:56:26 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3Dc506be7d9be5= 5721a44d38782309d8f7fcd5e99e commit c506be7d9be55721a44d38782309d8f7fcd5e99e Author: Maciej W. Rozycki Date: Fri Oct 21 08:54:18 2022 +0100 GDB/Python: Make `None' stand for `unlimited' in setting integer parame= ters =20 Similarly to booleans and following the fix for PR python/29217 make `gdb.parameter' accept `None' for `unlimited' with parameters of the PARAM_UINTEGER, PARAM_INTEGER, and PARAM_ZUINTEGER_UNLIMITED types, as `None' is already returned by parameters of the two former types, so one might expect to be able to feed it back. It also makes it possible to avoid the need to know what the internal integer representation is for the special setting of `unlimited'. =20 Expand the testsuite accordingly. =20 Approved-By: Simon Marchi Diff: --- gdb/python/lib/gdb/__init__.py | 9 ++++++--- gdb/python/py-param.c | 10 +++++++--- gdb/testsuite/gdb.python/py-parameter.exp | 21 +++++++++++++++------ 3 files changed, 28 insertions(+), 12 deletions(-) diff --git a/gdb/python/lib/gdb/__init__.py b/gdb/python/lib/gdb/__init__.py index 191915e4f3b..dab57a3102f 100644 --- a/gdb/python/lib/gdb/__init__.py +++ b/gdb/python/lib/gdb/__init__.py @@ -235,9 +235,12 @@ def find_pc_line(pc): =20 def set_parameter(name, value): """Set the GDB parameter NAME to VALUE.""" - # Handle the specific case of booleans here, because gdb.parameter - # can return them, but they can't be passed to 'set' this way. - if isinstance(value, bool): + # Handle the specific cases of None and booleans here, because + # gdb.parameter can return them, but they can't be passed to 'set' + # this way. + if value is None: + value =3D "unlimited" + elif isinstance(value, bool): if value: value =3D "on" else: diff --git a/gdb/python/py-param.c b/gdb/python/py-param.c index 5d509ba4658..cab0c563b0e 100644 --- a/gdb/python/py-param.c +++ b/gdb/python/py-param.c @@ -243,14 +243,18 @@ set_parameter_value (parmpy_object *self, PyObject *v= alue) long l; int ok; =20 - if (!PyLong_Check (value)) + if (value =3D=3D Py_None + && (self->type =3D=3D var_uinteger || self->type =3D=3D var_integer)) + l =3D 0; + else if (value =3D=3D Py_None && self->type =3D=3D var_zuinteger_unlimite= d) + l =3D -1; + else if (!PyLong_Check (value)) { PyErr_SetString (PyExc_RuntimeError, _("The value must be integer.")); return -1; } - - if (! gdb_py_int_as_long (value, &l)) + else if (! gdb_py_int_as_long (value, &l)) return -1; =20 switch (self->type) diff --git a/gdb/testsuite/gdb.python/py-parameter.exp b/gdb/testsuite/gdb.= python/py-parameter.exp index 3476d4344c2..56ae2a525a9 100644 --- a/gdb/testsuite/gdb.python/py-parameter.exp +++ b/gdb/testsuite/gdb.python/py-parameter.exp @@ -346,18 +346,21 @@ proc_with_prefix test_gdb_parameter { } { "listsize" { set param_get_zero None set param_get_minus_one -1 + set param_get_none None set param_get_unlimited None set param_set_minus_one "" } "print elements" { set param_get_zero None set param_get_minus_one None + set param_get_none None set param_get_unlimited None set param_set_minus_one $param_range_error } "max-completions" { set param_get_zero 0 set param_get_minus_one -1 + set param_get_none -1 set param_get_unlimited -1 set param_set_minus_one "" } @@ -385,6 +388,12 @@ proc_with_prefix test_gdb_parameter { } { gdb_test "python print(gdb.parameter('$param'))" \ $param_get_minus_one "test value of -1" =20 + gdb_test_no_output "python gdb.set_parameter('$param', None)" \ + "test set to None" + + gdb_test "python print(gdb.parameter('$param'))" \ + $param_get_none "test value of None" + gdb_test_no_output "python gdb.set_parameter('$param', 'unlimited')" \ "test set to 'unlimited'" =20 @@ -425,19 +434,19 @@ proc_with_prefix test_integer_parameter { } { set param_get_zero None set param_get_minus_one None set param_get_minus_five 1 - set param_get_none 5 + set param_get_none None set param_set_minus_one $param_range_error set param_set_minus_five $param_range_error - set param_set_none $param_integer_error + set param_set_none "" } PARAM_INTEGER { set param_get_zero None set param_get_minus_one -1 set param_get_minus_five -5 - set param_get_none 5 + set param_get_none None set param_set_minus_one -1 set param_set_minus_five -5 - set param_set_none $param_integer_error + set param_set_none "" } PARAM_ZINTEGER { set param_get_zero 0 @@ -461,10 +470,10 @@ proc_with_prefix test_integer_parameter { } { set param_get_zero 0 set param_get_minus_one -1 set param_get_minus_five 1 - set param_get_none 5 + set param_get_none -1 set param_set_minus_one "" set param_set_minus_five $param_range_error - set param_set_none $param_integer_error + set param_set_none "" } default { error "invalid kind: $kind"