From patchwork Thu Mar 21 18:10:32 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 87465 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 5678B3858286 for ; Thu, 21 Mar 2024 18:11:12 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mail-io1-xd2d.google.com (mail-io1-xd2d.google.com [IPv6:2607:f8b0:4864:20::d2d]) by sourceware.org (Postfix) with ESMTPS id 7281C3858C98 for ; Thu, 21 Mar 2024 18:10:34 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 7281C3858C98 Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 7281C3858C98 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::d2d ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1711044637; cv=none; b=nD9SDRzLSCIZ3229l1XDnsx3KslDL18bsqcUovQy+L5pODf2jRjjGLv3AWPy5gDayByYD185/yGgJSQIZlf9bvmYny8sCRezqpnI4eOzcCPfjsgEL5iYhuamEYz4sZUQWxYx9AYfK1bmJJnP0Sbyo4o4y6hS/J8kYD5rtsxuPXY= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1711044637; c=relaxed/simple; bh=YWXqDbcTwxcNHRuLuL9aa4f8FxrV4/0ItdLUKuEU40k=; h=DKIM-Signature:From:Date:Subject:MIME-Version:Message-Id:To; b=yAmzUhZYlIBVPxwin/BrFuKi65DIJEf+WrTttseXHZ98bPeAqYG4FhHs8Tn7E2PYuYe8jioT9aOzLzgX33tt8FfZ2fuNhyhniZoTXDMImrabZCkgMLS1VzamPM9gI7xYuLgXUQpJX04Wf+t+ONJDDY8TCs3w0QdVx2O9+Uvlt3A= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-io1-xd2d.google.com with SMTP id ca18e2360f4ac-7cc0e0dba0fso57549739f.3 for ; Thu, 21 Mar 2024 11:10:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1711044633; x=1711649433; darn=sourceware.org; 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=Mkk4q1RTngNzpu/Z1SO5ct5LVIzhkfWXw3iCQRmvGIw=; b=MX6UV58GGYVzyH5qiTM1vCp4QkqUiZvb3nhRRLGG/yr2agj3B5BYFa5oBGxMHcSB4A 5eIf80eQ7HY972r4LJ4AhALAWQBAtVZZp24uddq4sH0Q2IRCp2uxnVnBfSF0Kyq9XKb2 b/YV76TuxhQ/4WAk3BPPTmibQku/mQAoRLJainl7Fa4ndqN6RO30FT8SqhfFqZB60D9W fr5Snkm6mbhF67zDjBsh6gQ3WFX2rmuHvfPKgxX1MBbBuZPqAtqHcjY8PlCbVxAKkUhc /5PWJP8iPzFl13+rC9CXbmR++uZkKzWPFjNj4iMd2/rc6jFe7o1oCg9leToZW0E0Lz7E ESuA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711044633; x=1711649433; 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=Mkk4q1RTngNzpu/Z1SO5ct5LVIzhkfWXw3iCQRmvGIw=; b=O6OBF7B6uJZrCMPsE24GFhpTlnu9ekgkH7ycL9m1qD1Mllp273qHz0qsdZ894//Pyo 0RBbis8VKP8k3+cUU7fWhx+BPGb3/aIVpK2K3b0+Qcp5hpq85KLWn8xg9gPnJPp3lHlT c1FUEdNFMd1MbCMatKlJA0XLSP4YpECVh1hjAuU/TOFBAVgbxoNaiLYypARdXArER9Iy 6MS3SGkmcWN+BBM8DQHPe4NkfRykuEoNiIKmfOlAVYHYCP+4Fp11z94MsKnVBJM/56dv 7diVoxkYtRD4eDF2Mmz3zcmYECXot5j0YfEkPr+j/zeXS9KbFVQ5Vv1+ZEjFwYNf9dkf X6Nw== X-Gm-Message-State: AOJu0YzytDkgD+bh7zDh+6MaPYK3wODxz5Riyf1WAdlFhZiYnPCdwAZy a6qb1FiubZNrslFeeDlnFTXhL1tlLZPWtM15YTW+5wPntt6FwG+emvsKTJeopkn0whpOwVGVTVA = X-Google-Smtp-Source: AGHT+IF5XYDdqvXScxjNx8jjLzxY7ImFdtWInXNtCMDv7ZGVS55Exy5KHkKwS62ym+zwwoXJE3Lt+A== X-Received: by 2002:a6b:7209:0:b0:7d0:22b7:5e81 with SMTP id n9-20020a6b7209000000b007d022b75e81mr267671ioc.1.1711044633683; Thu, 21 Mar 2024 11:10:33 -0700 (PDT) Received: from localhost.localdomain (97-122-82-115.hlrn.qwest.net. [97.122.82.115]) by smtp.gmail.com with ESMTPSA id r23-20020a5e9517000000b007cf05417adbsm76582ioj.22.2024.03.21.11.10.33 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 21 Mar 2024 11:10:33 -0700 (PDT) From: Tom Tromey Date: Thu, 21 Mar 2024 12:10:32 -0600 Subject: [PATCH 1/3] Constify get_disassembler_options MIME-Version: 1.0 Message-Id: <20240321-disassemble-cleanup-v1-1-77448506f4ab@adacore.com> References: <20240321-disassemble-cleanup-v1-0-77448506f4ab@adacore.com> In-Reply-To: <20240321-disassemble-cleanup-v1-0-77448506f4ab@adacore.com> To: gdb-patches@sourceware.org X-Mailer: b4 0.12.4 X-Spam-Status: No, score=-11.2 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 autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org This changes get_disassembler_options to return a const char *. --- gdb/arm-tdep.c | 2 +- gdb/disasm.c | 2 +- gdb/disasm.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index 3b4ae15df07..82b06f63fcb 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -9640,7 +9640,7 @@ show_disassembly_style_sfunc (struct ui_file *file, int from_tty, struct cmd_list_element *c, const char *value) { struct gdbarch *gdbarch = get_current_arch (); - char *options = get_disassembler_options (gdbarch); + const char *options = get_disassembler_options (gdbarch); const char *style = ""; int len = 0; const char *opt; diff --git a/gdb/disasm.c b/gdb/disasm.c index 15d641cfc1f..5bab5cf6199 100644 --- a/gdb/disasm.c +++ b/gdb/disasm.c @@ -1291,7 +1291,7 @@ gdb_buffered_insn_length (struct gdbarch *gdbarch, return result; } -char * +const char * get_disassembler_options (struct gdbarch *gdbarch) { char **disassembler_options = gdbarch_disassembler_options (gdbarch); diff --git a/gdb/disasm.h b/gdb/disasm.h index a2251445707..9282632b270 100644 --- a/gdb/disasm.h +++ b/gdb/disasm.h @@ -385,7 +385,7 @@ extern int gdb_buffered_insn_length (struct gdbarch *gdbarch, /* Returns GDBARCH's disassembler options. */ -extern char *get_disassembler_options (struct gdbarch *gdbarch); +extern const char *get_disassembler_options (struct gdbarch *gdbarch); /* Sets the active gdbarch's disassembler options to OPTIONS. */ From patchwork Thu Mar 21 18:10:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 87466 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 6D6C238582A3 for ; Thu, 21 Mar 2024 18:11:13 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mail-io1-xd30.google.com (mail-io1-xd30.google.com [IPv6:2607:f8b0:4864:20::d30]) by sourceware.org (Postfix) with ESMTPS id 3874B3858C55 for ; Thu, 21 Mar 2024 18:10:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 3874B3858C55 Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 3874B3858C55 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::d30 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1711044638; cv=none; b=bct8GQ5opRnCpwuiz0gSK4F9cKUKfVtSoXDiARew+TRJreUu+f6bCB+apMFPXnwZ7pkTk7hI+73yzQsm1PcqxFWNYdpI4fwBUA1gy3twQOj9t4Gb1wDbdHpsvqg1ct8W9uak8lqOx10gCi+duy4oVnxeCQo2NoYWe14Xi0DHb4s= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1711044638; c=relaxed/simple; bh=0EYEapxSNQF/x0yIre+lAv3DTJh7Deh56vuINAxNg9Y=; h=DKIM-Signature:From:Date:Subject:MIME-Version:Message-Id:To; b=Y7UddBfFlwR7oAMEIPczUFG1plrQ5S3T6LlYOq/w6wLs8nHJuwiLNxyIF+mdD49CRs1gMAmgY5xtxYgRyxT4btQ6gGSDiD4/M7FfB9J/QMMG3ArL/j/F2xgWWYfKDE26gbFUDo8+rX67St1qa3n8jLpF9ufLT8ybzsWHiwbMihw= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-io1-xd30.google.com with SMTP id ca18e2360f4ac-7c8b95f7f73so37570939f.1 for ; Thu, 21 Mar 2024 11:10:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1711044634; x=1711649434; darn=sourceware.org; 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=/nhug5jhzc+yJV+QqIfQPLPbfbk/QjKkbwhgPjgciPg=; b=Xskl7lFdVbVSQ/6LRPheKib7EbNKXU6ITG04xZOKFCBu+QmghM/eFqrFg4jNNyGxEH sJMTP6BiDYpvObiFHTalGRrDF+6G2bZT73BknAgonh1OVaIPNF/5sN72E/jj7Hg/ASYe ciPK3BjqXjgUm/t3oUV+G/Gr3pHw1DEoRIj4mUzs9gwr269PyzuVSuQ+FgDIx4k5IC89 6boX4R0i7c0nXmy7iiEI0sNBmilDLUtSd3yr8afNK1fzFSALpidYNxlkmA3TvUgMHQL+ 6An+uYP8pZU//Z6wxp4jUUkIAIY5QzY4V8DELcWQU3X5cxfW6hKpFp6Xu1WBoIOTyKfv trpA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711044634; x=1711649434; 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=/nhug5jhzc+yJV+QqIfQPLPbfbk/QjKkbwhgPjgciPg=; b=q7J2XXIJpdWBIwUyPhJNbVgEBw2Aj7S7p8Bey4xdNn05DLk00J9mfRhlME5ojDmyCy ys76aVnU11UDMTt7SqnyiLRKnd2LSq/VxHjmspf5HqoDfU6TZYQgBqtTqM8bIsC7VBLd lC5OnIZXK3qJ56suCn4lovx36k/ba9aAuHVC8EhYyu16PQujBQKLnuinABoHf2OPrZNZ uDZ2uHHl6I3QRn5hA5jsDL742ISBysbJxdTvu+mCr29EDgcRXQtQGOo585C5qmXH5GBu +JXNrRWnsN/4098iV8jdHQw7qjteYXRS1mu7k7uVPU4yRFvqPPxrJ+9ZI66zWd9aYwrX WOPQ== X-Gm-Message-State: AOJu0Yz7ZqHUx3gLiwDqoLD0mRHDqicNAYnj5diMvgyTvLfcSCQrAyYe WwF1OrpcmrBML5iVCiCEjIMV68JcstkGMevhd7+Tsi+yuEngn3CoCfyRxQZ/GSXKRJOEKoN3W44 = X-Google-Smtp-Source: AGHT+IHCCtqe+VFsv5D67COKTZ0vfXldBFXmVzKFBnn2lDZPeVdLwlhm8TNAqpqL/YLIIJ/XmNGIkA== X-Received: by 2002:a6b:6e05:0:b0:7cc:ea7:4f01 with SMTP id d5-20020a6b6e05000000b007cc0ea74f01mr2544146ioh.8.1711044634305; Thu, 21 Mar 2024 11:10:34 -0700 (PDT) Received: from localhost.localdomain (97-122-82-115.hlrn.qwest.net. [97.122.82.115]) by smtp.gmail.com with ESMTPSA id r23-20020a5e9517000000b007cf05417adbsm76582ioj.22.2024.03.21.11.10.33 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 21 Mar 2024 11:10:33 -0700 (PDT) From: Tom Tromey Date: Thu, 21 Mar 2024 12:10:33 -0600 Subject: [PATCH 2/3] Remove some unnecessary casts MIME-Version: 1.0 Message-Id: <20240321-disassemble-cleanup-v1-2-77448506f4ab@adacore.com> References: <20240321-disassemble-cleanup-v1-0-77448506f4ab@adacore.com> In-Reply-To: <20240321-disassemble-cleanup-v1-0-77448506f4ab@adacore.com> To: gdb-patches@sourceware.org X-Mailer: b4 0.12.4 X-Spam-Status: No, score=-11.2 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 autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org I found a few unnecessary casts when calling set_gdbarch_disassembler_options_implicit. --- gdb/mips-tdep.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c index 8339d2f1d4c..107d06191b4 100644 --- a/gdb/mips-tdep.c +++ b/gdb/mips-tdep.c @@ -8736,14 +8736,14 @@ mips_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) set_gdbarch_print_insn (gdbarch, gdb_print_insn_mips); if (mips_abi == MIPS_ABI_N64) - set_gdbarch_disassembler_options_implicit - (gdbarch, (const char *) mips_disassembler_options_n64); + set_gdbarch_disassembler_options_implicit (gdbarch, + mips_disassembler_options_n64); else if (mips_abi == MIPS_ABI_N32) - set_gdbarch_disassembler_options_implicit - (gdbarch, (const char *) mips_disassembler_options_n32); + set_gdbarch_disassembler_options_implicit (gdbarch, + mips_disassembler_options_n32); else - set_gdbarch_disassembler_options_implicit - (gdbarch, (const char *) mips_disassembler_options_o32); + set_gdbarch_disassembler_options_implicit (gdbarch, + mips_disassembler_options_o32); set_gdbarch_disassembler_options (gdbarch, &mips_disassembler_options); set_gdbarch_valid_disassembler_options (gdbarch, disassembler_options_mips ()); From patchwork Thu Mar 21 18:10:34 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 87467 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 919DB385829A for ; Thu, 21 Mar 2024 18:11:15 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mail-io1-xd30.google.com (mail-io1-xd30.google.com [IPv6:2607:f8b0:4864:20::d30]) by sourceware.org (Postfix) with ESMTPS id AA8923858D28 for ; Thu, 21 Mar 2024 18:10:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org AA8923858D28 Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org AA8923858D28 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::d30 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1711044638; cv=none; b=sHONFpGE01Zth/eu/SKrbhrJoJKdKjl0c66Pf2XflX8ItwSJZJBB0HPDGhzh2Di3YAY1QpBY1cGhnl1bXdSQ1Qch69WeB/HIqemk1p1/4z1B3avzeFAjSyFPaz/cxOjdCu4M/PP5Wb9HvfeFqk+lp4dQ0Lr92gerN90Rv/qVQU0= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1711044638; c=relaxed/simple; bh=jCqj0MkVhdtlLoeTwdZ+V+NLEDn8c2LfT4u35PXJmKI=; h=DKIM-Signature:From:Date:Subject:MIME-Version:Message-Id:To; b=lWygq1ZS74G19l3+cwkwHGiXLBsQH00zJ4wOhfU9iSIervZPYy84ZVxSIKuiJVFu7roet2MxTSUjwj96AN2s7v3QMuJA2FdTSvkpuLbxX4xn2YrNwn5jEVFu0b9CAyNtqP5A/jtC8YtRIj8e/jm9Wp3bk75OhpgQ0OCQmNWCpOk= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-io1-xd30.google.com with SMTP id ca18e2360f4ac-7cf25c1ee7aso52925139f.2 for ; Thu, 21 Mar 2024 11:10:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1711044635; x=1711649435; darn=sourceware.org; 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=yFPeO/t5nCKesMX1oEwIsOVYVXhw4sMZWkdoPFxQzkA=; b=ENKWGn5m9qrcrLbdA5TXK7OyFTxfYRLsTtu47/eaOzOm0vpyEYpfNS39IfSw8a+afZ uwXq8l0cawguA1TZewfAuG9gjyhX3DJzkwXpqA7PxLRqHcW+pTtWhw3A9vvjd+Jq4jZT pIULBe2p0bO1J8cIYS72gAjjL+eqg3fDbwR/xPmtGUzyATESHqU999h/R6xEEMbYj5FW 2mjqjmCALgnAplQgQYm/QVufACs/R2A1zf81lltBhw0/iD4tUpyMXev2Pnmfrg6wlzyW rRJaGibSsk010AdRwuls31XzefUjwNsUBmxAiiQn1me6/T5eMD9bAs3WsKLo8c8I+HGI q7Bw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711044635; x=1711649435; 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=yFPeO/t5nCKesMX1oEwIsOVYVXhw4sMZWkdoPFxQzkA=; b=gcHvAqpm1jNKPuBJJIu9Rn8DaUi/LXyT0ObFBAqvUHx1gEQnPpZajETAZWKMmTN+n3 XrSjBl66/4W7DKRpdftXd/hznkk031ESWoSwss46TY7MDntZwCjp0QmJ0BoPOjvFqKuT zd/lt9rNLGpSACmdwxd07vAQtM4MqozFazw9zCdTAvQB/G+iUIRt8TK07RsGfJXsYp+C PaAd8q84JQKdWLxqI98CHSzpLSfTXzdecWa/dKEfAGCKOnoMkdXuuH20xnTmU5tc8d/0 JFt5btDRfxqt3Q0+OEATMJMtdxSFr2E8UNrcBqKw0RMSu1cS9gEsu5/oZNHc9VetnUZD 5S6A== X-Gm-Message-State: AOJu0YwBnWrk61WO4zRUKeKHCWt8hIxnytKWjoNsyGO47eznyMkFx8IT bnX4E5/OPwxwMkYv8Ksd5QD4UXJh+wIVFUG2bKXG5AEYS8NFLGpUY1UKu6Eg2uRjzy/7qQI/AfQ = X-Google-Smtp-Source: AGHT+IF7MvYhL9F3YtWQPtajNA9Q4LT6RMEbMxs0Zp5zws7fI/RPyaonuMKsJ70FK6iBBXTCfH/YSw== X-Received: by 2002:a6b:7d0a:0:b0:7cb:e510:c289 with SMTP id c10-20020a6b7d0a000000b007cbe510c289mr207493ioq.9.1711044634689; Thu, 21 Mar 2024 11:10:34 -0700 (PDT) Received: from localhost.localdomain (97-122-82-115.hlrn.qwest.net. [97.122.82.115]) by smtp.gmail.com with ESMTPSA id r23-20020a5e9517000000b007cf05417adbsm76582ioj.22.2024.03.21.11.10.34 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 21 Mar 2024 11:10:34 -0700 (PDT) From: Tom Tromey Date: Thu, 21 Mar 2024 12:10:34 -0600 Subject: [PATCH 3/3] Use std::string for disassembler options MIME-Version: 1.0 Message-Id: <20240321-disassemble-cleanup-v1-3-77448506f4ab@adacore.com> References: <20240321-disassemble-cleanup-v1-0-77448506f4ab@adacore.com> In-Reply-To: <20240321-disassemble-cleanup-v1-0-77448506f4ab@adacore.com> To: gdb-patches@sourceware.org X-Mailer: b4 0.12.4 X-Spam-Status: No, score=-11.2 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 autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org I noticed that the disassembler_options code uses manual memory management. It seemed simpler to replace this with std::string. --- gdb/arc-tdep.c | 15 +++++++-------- gdb/arch-utils.c | 6 +++--- gdb/arm-tdep.c | 4 ++-- gdb/disasm.c | 20 ++++++++------------ gdb/gdbarch-gen.h | 4 ++-- gdb/gdbarch.c | 6 +++--- gdb/gdbarch_components.py | 2 +- gdb/mips-tdep.c | 2 +- gdb/riscv-tdep.c | 2 +- gdb/rs6000-tdep.c | 2 +- gdb/s390-tdep.c | 2 +- 11 files changed, 30 insertions(+), 35 deletions(-) diff --git a/gdb/arc-tdep.c b/gdb/arc-tdep.c index 7dd43cc239f..63a1fd918d1 100644 --- a/gdb/arc-tdep.c +++ b/gdb/arc-tdep.c @@ -299,7 +299,7 @@ static const struct arc_register_feature arc_common_aux_reg_feature = } }; -static char *arc_disassembler_options = NULL; +static std::string arc_disassembler_options; /* Functions are sorted in the order as they are used in the _initialize_arc_tdep (), which uses the same order as gdbarch.h. Static @@ -2365,7 +2365,6 @@ arc_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) = tdesc_architecture (info.target_desc); if (tdesc_arch != NULL) { - xfree (arc_disassembler_options); /* FIXME: It is not really good to change disassembler options behind the scene, because that might override options specified by the user. However as of now ARC doesn't support @@ -2386,24 +2385,24 @@ arc_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) switch (tdesc_arch->mach) { case bfd_mach_arc_arc601: - arc_disassembler_options = xstrdup ("cpu=arc601"); + arc_disassembler_options = "cpu=arc601"; break; case bfd_mach_arc_arc600: - arc_disassembler_options = xstrdup ("cpu=arc600"); + arc_disassembler_options = "cpu=arc600"; break; case bfd_mach_arc_arc700: - arc_disassembler_options = xstrdup ("cpu=arc700"); + arc_disassembler_options = "cpu=arc700"; break; case bfd_mach_arc_arcv2: /* Machine arcv2 has three arches: ARCv2, EM and HS; where ARCv2 is treated as EM. */ if (arc_arch_is_hs (tdesc_arch)) - arc_disassembler_options = xstrdup ("cpu=hs38_linux"); + arc_disassembler_options = "cpu=hs38_linux"; else - arc_disassembler_options = xstrdup ("cpu=em4_fpuda"); + arc_disassembler_options = "cpu=em4_fpuda"; break; default: - arc_disassembler_options = NULL; + arc_disassembler_options = ""; break; } } diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c index ae3354f6579..d404d1fc38a 100644 --- a/gdb/arch-utils.c +++ b/gdb/arch-utils.c @@ -1156,11 +1156,11 @@ pstring (const char *string) } static const char * -pstring_ptr (char **string) +pstring_ptr (std::string *string) { - if (string == NULL || *string == NULL) + if (string == nullptr) return "(null)"; - return *string; + return string->c_str (); } /* Helper function to print a list of strings, represented as "const diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index 82b06f63fcb..3a5fbe71b8b 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -232,7 +232,7 @@ static const char *const arm_register_names[] = "fps", "cpsr" }; /* 24 25 */ /* Holds the current set of options to be passed to the disassembler. */ -static char *arm_disassembler_options; +static std::string arm_disassembler_options; /* Valid register name styles. */ static const char **valid_disassembly_styles; @@ -11042,7 +11042,7 @@ _initialize_arm_tdep () &setarmcmdlist, &showarmcmdlist, &setlist, &showlist); - arm_disassembler_options = xstrdup ("reg-names-std"); + arm_disassembler_options = "reg-names-std"; const disasm_options_t *disasm_options = &disassembler_options_arm ()->options; int num_disassembly_styles = 0; diff --git a/gdb/disasm.c b/gdb/disasm.c index 5bab5cf6199..cafbc078c60 100644 --- a/gdb/disasm.c +++ b/gdb/disasm.c @@ -1294,17 +1294,17 @@ gdb_buffered_insn_length (struct gdbarch *gdbarch, const char * get_disassembler_options (struct gdbarch *gdbarch) { - char **disassembler_options = gdbarch_disassembler_options (gdbarch); - if (disassembler_options == NULL) - return NULL; - return *disassembler_options; + std::string *disassembler_options = gdbarch_disassembler_options (gdbarch); + if (disassembler_options == nullptr || disassembler_options->empty ()) + return nullptr; + return disassembler_options->c_str (); } void set_disassembler_options (const char *prospective_options) { struct gdbarch *gdbarch = get_current_arch (); - char **disassembler_options = gdbarch_disassembler_options (gdbarch); + std::string *disassembler_options = gdbarch_disassembler_options (gdbarch); const disasm_options_and_args_t *valid_options_and_args; const disasm_options_t *valid_options; gdb::unique_xmalloc_ptr prospective_options_local @@ -1317,11 +1317,8 @@ set_disassembler_options (const char *prospective_options) to reset their disassembler options to NULL. */ if (options == NULL) { - if (disassembler_options != NULL) - { - free (*disassembler_options); - *disassembler_options = NULL; - } + if (disassembler_options != nullptr) + disassembler_options->clear (); return; } @@ -1373,8 +1370,7 @@ set_disassembler_options (const char *prospective_options) } } - free (*disassembler_options); - *disassembler_options = xstrdup (options); + *disassembler_options = options; } static void diff --git a/gdb/gdbarch-gen.h b/gdb/gdbarch-gen.h index 13dd0ed5d5e..ebcff80bb9e 100644 --- a/gdb/gdbarch-gen.h +++ b/gdb/gdbarch-gen.h @@ -1715,8 +1715,8 @@ extern void set_gdbarch_addressable_memory_unit_size (struct gdbarch *gdbarch, g extern const char * gdbarch_disassembler_options_implicit (struct gdbarch *gdbarch); extern void set_gdbarch_disassembler_options_implicit (struct gdbarch *gdbarch, const char * disassembler_options_implicit); -extern char ** gdbarch_disassembler_options (struct gdbarch *gdbarch); -extern void set_gdbarch_disassembler_options (struct gdbarch *gdbarch, char ** disassembler_options); +extern std::string * gdbarch_disassembler_options (struct gdbarch *gdbarch); +extern void set_gdbarch_disassembler_options (struct gdbarch *gdbarch, std::string * disassembler_options); extern const disasm_options_and_args_t * gdbarch_valid_disassembler_options (struct gdbarch *gdbarch); extern void set_gdbarch_valid_disassembler_options (struct gdbarch *gdbarch, const disasm_options_and_args_t * valid_disassembler_options); diff --git a/gdb/gdbarch.c b/gdb/gdbarch.c index 80a04bf0caf..9319571deba 100644 --- a/gdb/gdbarch.c +++ b/gdb/gdbarch.c @@ -252,7 +252,7 @@ struct gdbarch gdbarch_gnu_triplet_regexp_ftype *gnu_triplet_regexp = default_gnu_triplet_regexp; gdbarch_addressable_memory_unit_size_ftype *addressable_memory_unit_size = default_addressable_memory_unit_size; const char * disassembler_options_implicit = 0; - char ** disassembler_options = 0; + std::string * disassembler_options = 0; const disasm_options_and_args_t * valid_disassembler_options = 0; gdbarch_type_align_ftype *type_align = default_type_align; gdbarch_get_pc_address_flags_ftype *get_pc_address_flags = default_get_pc_address_flags; @@ -5362,7 +5362,7 @@ set_gdbarch_disassembler_options_implicit (struct gdbarch *gdbarch, gdbarch->disassembler_options_implicit = disassembler_options_implicit; } -char ** +std::string * gdbarch_disassembler_options (struct gdbarch *gdbarch) { gdb_assert (gdbarch != NULL); @@ -5374,7 +5374,7 @@ gdbarch_disassembler_options (struct gdbarch *gdbarch) void set_gdbarch_disassembler_options (struct gdbarch *gdbarch, - char ** disassembler_options) + std::string * disassembler_options) { gdbarch->disassembler_options = disassembler_options; } diff --git a/gdb/gdbarch_components.py b/gdb/gdbarch_components.py index 762d48a0187..7d913ade621 100644 --- a/gdb/gdbarch_components.py +++ b/gdb/gdbarch_components.py @@ -2711,7 +2711,7 @@ Functions for allowing a target to modify its disassembler options. ) Value( - type="char **", + type="std::string *", name="disassembler_options", invalid=False, printer="pstring_ptr (gdbarch->disassembler_options)", diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c index 107d06191b4..67bed1d2ba0 100644 --- a/gdb/mips-tdep.c +++ b/gdb/mips-tdep.c @@ -213,7 +213,7 @@ struct target_desc *mips_tdesc_gp32; struct target_desc *mips_tdesc_gp64; /* The current set of options to be passed to the disassembler. */ -static char *mips_disassembler_options; +static std::string mips_disassembler_options; /* Implicit disassembler options for individual ABIs. These tell libopcodes to use general-purpose register names corresponding diff --git a/gdb/riscv-tdep.c b/gdb/riscv-tdep.c index 68ef3233e8d..604a19fe2a6 100644 --- a/gdb/riscv-tdep.c +++ b/gdb/riscv-tdep.c @@ -133,7 +133,7 @@ static const char *riscv_feature_name_virtual = "org.gnu.gdb.riscv.virtual"; static const char *riscv_feature_name_vector = "org.gnu.gdb.riscv.vector"; /* The current set of options to be passed to the disassembler. */ -static char *riscv_disassembler_options; +static std::string riscv_disassembler_options; /* Cached information about a frame. */ diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c index c8450345be2..3157213cb1f 100644 --- a/gdb/rs6000-tdep.c +++ b/gdb/rs6000-tdep.c @@ -130,7 +130,7 @@ && (regnum) < (tdep)->ppc_cefpr0_regnum + ppc_num_efprs) /* Holds the current set of options to be passed to the disassembler. */ -static char *powerpc_disassembler_options; +static std::string powerpc_disassembler_options; /* The list of available "set powerpc ..." and "show powerpc ..." commands. */ diff --git a/gdb/s390-tdep.c b/gdb/s390-tdep.c index 75983550a5c..174338e1885 100644 --- a/gdb/s390-tdep.c +++ b/gdb/s390-tdep.c @@ -45,7 +45,7 @@ #include "features/s390x-linux64.c" /* Holds the current set of options to be passed to the disassembler. */ -static char *s390_disassembler_options; +static std::string s390_disassembler_options; /* Breakpoints. */