gdb/dap: ignore unused keyword args in step_out
authorGregory Anders <greg@gpanders.com>
Fri, 1 Sep 2023 21:02:18 +0000 (16:02 -0500)
committerTom Tromey <tromey@adacore.com>
Wed, 20 Sep 2023 16:59:38 +0000 (10:59 -0600)
Some DAP clients may send additional parameters in the stepOut command
(e.g. "granularity") which are not used by GDB, but should nonetheless
be accepted without error.

Approved-By: Tom Tromey <tom@tromey.com>
gdb/python/lib/gdb/dap/next.py
gdb/python/lib/gdb/dap/server.py

index 5046963cb8137384d92964c22b9fdc5f7995a202..e5bb8d64da0953e259c3335ab57bf6ee76d88726 100644 (file)
@@ -74,7 +74,7 @@ def step_in(
 
 
 @request("stepOut")
-def step_out(*, threadId: int, singleThread: bool = False):
+def step_out(*, threadId: int, singleThread: bool = False, **args):
     send_gdb(lambda: _handle_thread_step(threadId, singleThread, True))
     send_gdb(ExecutionInvoker("finish", StopKinds.STEP))
 
index db7893a387b95c39db6f7fa90c2dd2ed8008ff0b..d84bca5d1fc9cbc8e7e619f0007fca75c36c35b6 100644 (file)
@@ -13,6 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
+import inspect
 import json
 import queue
 import sys
@@ -165,6 +166,12 @@ def request(name):
 
     def wrap(func):
         global _commands
+        code = func.__code__
+        # We don't permit requests to have positional arguments.
+        assert code.co_posonlyargcount == 0
+        assert code.co_argcount == 0
+        # A request must have a **args parameter.
+        assert code.co_flags & inspect.CO_VARKEYWORDS
         # All requests must run in the DAP thread.
         # Also type-check the calls.
         func = in_dap_thread(type_check(func))