From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io1-xd29.google.com (mail-io1-xd29.google.com [IPv6:2607:f8b0:4864:20::d29]) by sourceware.org (Postfix) with ESMTPS id 8871D3858C5F for ; Thu, 18 May 2023 20:18:11 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 8871D3858C5F Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com Received: by mail-io1-xd29.google.com with SMTP id ca18e2360f4ac-76fe17da95fso57535639f.2 for ; Thu, 18 May 2023 13:18:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1684441091; x=1687033091; h=to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=i1nQ9t2omyjzsFaoKmKyo74/O9KoF4r3iMELs4EGkOI=; b=BOW5nZ/I3rjchTrOtzaFN4M9TZaodYlvhZyvOHaPyTp0UeWexRBTUyN3djf4SveU+P 9oCHV3FMIONIiE8RYFW120+oSvEkDf3SouJhGkWHj2MGV1vYUjIgRfSFvpVVeoH8HcYx sqyYV+uiWcMzFjBjlZcjZeqxMeed7pB9UysLDHV6GD6trfZEyxXDF3tNUu1LGAEb3DqK 7h4Z10f7h/B3rjvxRC+S72utYOs0uY6EAymzTA/pSFron+4dzQMDjLFGJAj6Urb5o0DM HPelmGGfMkNak2xFs9UDR5p9GM4C24An/nKphYjLvh3E+GTnOG5bStdlD7tWbKqPsMxk kaMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684441091; x=1687033091; h=to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=i1nQ9t2omyjzsFaoKmKyo74/O9KoF4r3iMELs4EGkOI=; b=Y8FpfkAlLcXvMol9kjH/R35HUsIBTZb4+RGPGSuyRZ6/H3JU4u8GLNhxlKFjuYvkBZ 2VfxBAXz54izN4iUoEXXom+eS+4DSPHdpc6+hXX9HSRX4rK/sEjcUMO6A8VJS92oPPMe K1Lkj4Iwc1V/gocDOE3arUqt5zWojBjOL1wQ51fyOpYwt2eJUF0OVicFZBcGMTY2zPwm 8uxz74EeJvJzp+uPpXLBJc7UYo62fEOGKHV6bWeB2xolo+DuHFVdS8quuPfrqTz5C+Oo v+FpSttUyTQDCBPX4NYgTlTxx6vLm98Uk0fI5Sxt/YGc+du3H1ll7xy8ClNl6MW6dskD dDuA== X-Gm-Message-State: AC+VfDwSCV+3AOABgo2dlJ/Wn1cPhRtOHgZEBTrwgB5XV/U/4388rPwB vN7y41I9s017buVCwzSNy2CXtuDqqKGRR9md9mJ4hw== X-Google-Smtp-Source: ACHHUZ5QBsJjS/wEQoeT9qW7zIGaP59YB9K5mWgIOO11pPVaPWyQs3LClPSxsMmPdJESw/hUzCCdtQ== X-Received: by 2002:a5d:9f06:0:b0:760:e308:107e with SMTP id q6-20020a5d9f06000000b00760e308107emr6940469iot.0.1684441090743; Thu, 18 May 2023 13:18:10 -0700 (PDT) Received: from localhost.localdomain (75-166-157-55.hlrn.qwest.net. [75.166.157.55]) by smtp.gmail.com with ESMTPSA id i9-20020a05663813c900b00418ba399842sm703087jaj.13.2023.05.18.13.18.10 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 18 May 2023 13:18:10 -0700 (PDT) From: Tom Tromey Date: Thu, 18 May 2023 14:18:09 -0600 Subject: [PATCH v2 3/9] Use accessor for mi_parse::args MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20230404-dap-loaded-sources-v2-3-93f229095e03@adacore.com> References: <20230404-dap-loaded-sources-v2-0-93f229095e03@adacore.com> In-Reply-To: <20230404-dap-loaded-sources-v2-0-93f229095e03@adacore.com> To: gdb-patches@sourceware.org X-Mailer: b4 0.12.2 X-Spam-Status: No, score=-11.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: This changes mi_parse::args to be a private member, retrieved via accessor. It also changes this member to be a std::string. This makes it simpler for a subsequent patch to implement different behavior for argument parsing. --- gdb/mi/mi-cmds.c | 6 +++--- gdb/mi/mi-main.c | 2 +- gdb/mi/mi-parse.c | 3 +-- gdb/mi/mi-parse.h | 13 ++++++++++++- gdb/python/py-micmd.c | 5 +++-- 5 files changed, 20 insertions(+), 9 deletions(-) diff --git a/gdb/mi/mi-cmds.c b/gdb/mi/mi-cmds.c index 284453db85d..ca8c633e218 100644 --- a/gdb/mi/mi-cmds.c +++ b/gdb/mi/mi-cmds.c @@ -49,11 +49,11 @@ struct mi_command_mi : public mi_command with arguments contained within PARSE. */ void invoke (struct mi_parse *parse) const override { - mi_parse_argv (parse->args, parse); + mi_parse_argv (parse->args (), parse); if (parse->argv == nullptr) error (_("Problem parsing arguments: %s %s"), parse->command, - parse->args); + parse->args ()); this->m_argv_function (parse->command, parse->argv, parse->argc); } @@ -87,7 +87,7 @@ struct mi_command_cli : public mi_command is passed through to the CLI function as its argument string. */ void invoke (struct mi_parse *parse) const override { - const char *args = m_args_p ? parse->args : nullptr; + const char *args = m_args_p ? parse->args () : nullptr; mi_execute_cli_command (m_cli_name, m_args_p, args); } diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c index 840663e0fb4..aaebce40fac 100644 --- a/gdb/mi/mi-main.c +++ b/gdb/mi/mi-main.c @@ -1813,7 +1813,7 @@ captured_mi_execute_command (struct ui_out *uiout, struct mi_parse *context) if (mi_debug_p) gdb_printf (gdb_stdlog, " token=`%s' command=`%s' args=`%s'\n", - context->token, context->command, context->args); + context->token, context->command, context->args ()); mi_cmd_execute (context); diff --git a/gdb/mi/mi-parse.c b/gdb/mi/mi-parse.c index bda554a568e..bf3b534e590 100644 --- a/gdb/mi/mi-parse.c +++ b/gdb/mi/mi-parse.c @@ -212,7 +212,6 @@ mi_parse::~mi_parse () { xfree (command); xfree (token); - xfree (args); freeargv (argv); } @@ -346,7 +345,7 @@ mi_parse (const char *cmd, char **token) } /* Save the rest of the arguments for the command. */ - parse->args = xstrdup (chp); + parse->set_args (chp); /* Fully parsed, flag as an MI command. */ parse->op = MI_COMMAND; diff --git a/gdb/mi/mi-parse.h b/gdb/mi/mi-parse.h index 75bb36b1c77..d4ac3f002e4 100644 --- a/gdb/mi/mi-parse.h +++ b/gdb/mi/mi-parse.h @@ -46,12 +46,19 @@ struct mi_parse DISABLE_COPY_AND_ASSIGN (mi_parse); + /* Return the full argument string, as used by commands which are + implemented as CLI commands. */ + const char *args () const + { return m_args.c_str (); } + + void set_args (const char *args) + { m_args = args; } + enum mi_command_type op = MI_COMMAND; char *command = nullptr; char *token = nullptr; const struct mi_command *cmd = nullptr; struct mi_timestamp *cmd_start = nullptr; - char *args = nullptr; char **argv = nullptr; int argc = 0; int all = 0; @@ -62,6 +69,10 @@ struct mi_parse /* The language that should be used to evaluate the MI command. Ignored if set to language_unknown. */ enum language language = language_unknown; + + private: + + std::string m_args; }; /* Attempts to parse CMD returning a ``struct mi_parse''. If CMD is diff --git a/gdb/python/py-micmd.c b/gdb/python/py-micmd.c index d7d95918cb4..88d52db2202 100644 --- a/gdb/python/py-micmd.c +++ b/gdb/python/py-micmd.c @@ -355,10 +355,11 @@ mi_command_py::invoke (struct mi_parse *parse) const pymicmd_debug_printf ("this = %p, name = %s", this, name ()); - mi_parse_argv (parse->args, parse); + mi_parse_argv (parse->args (), parse); if (parse->argv == nullptr) - error (_("Problem parsing arguments: %s %s"), parse->command, parse->args); + error (_("Problem parsing arguments: %s %s"), parse->command, + parse->args ()); gdbpy_enter enter_py; -- 2.40.0