* [PATCH v2 0/4] [gdb] Remove doxygen support
@ 2026-06-01 10:53 Tom de Vries
2026-06-01 10:53 ` [PATCH v2 1/4] [gdb/doc] " Tom de Vries
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Tom de Vries @ 2026-06-01 10:53 UTC (permalink / raw)
To: gdb-patches
This patch series removes doxygen support.
The first patch removes the ability to generate it.
The second, third and fourth patch remove doxygen-specific annotations.
Tested on x86_64-linux.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34186
Changes in v2:
- fixed nit in second patch
- added fourth patch
Versions:
- v1 https://sourceware.org/pipermail/gdb-patches/2026-May/227689.html
Tom de Vries (4):
[gdb/doc] Remove doxygen support
[gdb] Remove '/* *' multi-line comment marker
[gdb] Remove \page marker
[gdb] Remove '/**<' comment marker
gdb/cli/cli-script.h | 6 +-
gdb/defs.h | 70 +++----
gdb/doc/Doxyfile-base.in | 92 ---------
gdb/doc/Doxyfile-gdb-api.in | 39 ----
gdb/doc/Doxyfile-gdb-xref.in | 44 -----
gdb/doc/Doxyfile-gdbserver.in | 45 -----
gdb/doc/Makefile.in | 36 ----
gdb/doc/doxy-index.in | 76 --------
gdb/doc/filter-for-doxygen | 14 --
gdb/doc/filter-params.pl | 11 --
gdb/dwarf2/call-site.h | 30 +--
gdb/gdbtypes.h | 340 +++++++++++++++++-----------------
gdb/main.h | 2 +-
gdb/type-codes.def | 58 +++---
gdb/valprint.h | 2 +-
gdbsupport/common-types.h | 12 +-
gdbsupport/host-defs.h | 4 +-
17 files changed, 261 insertions(+), 620 deletions(-)
delete mode 100644 gdb/doc/Doxyfile-base.in
delete mode 100644 gdb/doc/Doxyfile-gdb-api.in
delete mode 100644 gdb/doc/Doxyfile-gdb-xref.in
delete mode 100644 gdb/doc/Doxyfile-gdbserver.in
delete mode 100644 gdb/doc/doxy-index.in
delete mode 100755 gdb/doc/filter-for-doxygen
delete mode 100644 gdb/doc/filter-params.pl
base-commit: 6c2f6391861d9bec3a983cc80f48e2777e582504
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/4] [gdb/doc] Remove doxygen support
2026-06-01 10:53 [PATCH v2 0/4] [gdb] Remove doxygen support Tom de Vries
@ 2026-06-01 10:53 ` Tom de Vries
2026-06-01 10:53 ` [PATCH v2 2/4] [gdb] Remove '/* *' multi-line comment marker Tom de Vries
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Tom de Vries @ 2026-06-01 10:53 UTC (permalink / raw)
To: gdb-patches
We have some form of doxygen support, allowing to run "make doxy" in
build/gdb/doc, but in the result I didn't find any references to gdbsupport
(moved to top-level in 2019), so I'm assuming this is unmaintained for a long
time now.
Remove it.
Reviewed-by: Kevin Buettner <kevinb@redhat.com>
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34186
---
gdb/doc/Doxyfile-base.in | 92 -----------------------------------
gdb/doc/Doxyfile-gdb-api.in | 39 ---------------
gdb/doc/Doxyfile-gdb-xref.in | 44 -----------------
gdb/doc/Doxyfile-gdbserver.in | 45 -----------------
gdb/doc/Makefile.in | 36 --------------
gdb/doc/doxy-index.in | 76 -----------------------------
gdb/doc/filter-for-doxygen | 14 ------
gdb/doc/filter-params.pl | 11 -----
8 files changed, 357 deletions(-)
delete mode 100644 gdb/doc/Doxyfile-base.in
delete mode 100644 gdb/doc/Doxyfile-gdb-api.in
delete mode 100644 gdb/doc/Doxyfile-gdb-xref.in
delete mode 100644 gdb/doc/Doxyfile-gdbserver.in
delete mode 100644 gdb/doc/doxy-index.in
delete mode 100755 gdb/doc/filter-for-doxygen
delete mode 100644 gdb/doc/filter-params.pl
diff --git a/gdb/doc/Doxyfile-base.in b/gdb/doc/Doxyfile-base.in
deleted file mode 100644
index 4fb0d93c94a..00000000000
--- a/gdb/doc/Doxyfile-base.in
+++ /dev/null
@@ -1,92 +0,0 @@
-# Copyright (C) 2014-2026 Free Software Foundation, Inc.
-
-# Base doxyfile for GDB.
-# This file is part of GDB.
-
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-# The definitions in this file are shared by several different
-# renditions of GDB documentation, and should reflect common
-# GDB practices and assumptions.
-
-# (Note that we are not following a common doxygen practice, which is
-# to include the entirety of doxygen's large default Doxyfile, and
-# edit it slightly for the project. Instead, these Doxyfile fragments
-# include only parameter settings that differ from the default.)
-
-PROJECT_NAME = "GDB"
-
-# Start out with an everything-is-documented assumption. (Later
-# on we may want to limit to only specific areas.)
-
-EXTRACT_ALL = YES
-
-# These are intended for GDB developers, so include anything flagged
-# "internal".
-
-INTERNAL_DOCS = YES
-
-# Always dig through subdirectories.
-
-RECURSIVE = YES
-
-INCLUDE_PATH = @srcdir@/../ @srcdir@/../common @srcdir@/../../include/
-
-# Exclude testsuite and other subdirectories that do not have any code
-# that goes into GDB or GDBserver.
-
-EXCLUDE = @srcdir@/../gdbserver/ \
- ../gdbserver/ \
- @srcdir@/../gnulib/ \
- ../build-gnulib/ \
- @srcdir@/../testsuite/ \
- ../testsuite/ \
- @srcdir@/../stubs/
-
-# Scrub out any stuff that might be a problem for Doxygen.
-
-INPUT_FILTER = @srcdir@/filter-for-doxygen
-
-# Comment this out (or set to YES) to see lots of finicky complaints.
-
-WARN_IF_DOC_ERROR = NO
-
-# By default, HTML will be generated.
-
-# We are missing javascript to make this work?
-#HTML_DYNAMIC_SECTIONS = YES
-
-# In 1.8 only?
-#HTML_INDEX_NUM_ENTRIES = 10
-
-# We never have a use for a LaTex version of this.
-
-GENERATE_LATEX = NO
-
-# We always want to get to sources easily.
-
-SOURCE_BROWSER = YES
-
-FORCE_LOCAL_INCLUDES = YES
-
-# We would like to have full macro expansion, but it's very slow.
-
-ENABLE_PREPROCESSING = YES
-#MACRO_EXPANSION = YES
-#EXPAND_ONLY_PREDEF = YES
-#PREDEFINED = __attribute__(x)= __extension__=
-
-# Suppress the huge volume of chatter.
-
-QUIET = YES
diff --git a/gdb/doc/Doxyfile-gdb-api.in b/gdb/doc/Doxyfile-gdb-api.in
deleted file mode 100644
index d6437239f73..00000000000
--- a/gdb/doc/Doxyfile-gdb-api.in
+++ /dev/null
@@ -1,39 +0,0 @@
-# Copyright (C) 2014-2026 Free Software Foundation, Inc.
-
-# Doxygen file for GDB internal API.
-# This file is part of GDB.
-
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-@INCLUDE = Doxyfile-base
-
-PROJECT_NAME = "GDB (API)"
-
-# Enumerate the files to process. In general these should be header
-# files with definitions of general interest, rather than ones that
-# are specific to a single target or configuration. (The cross
-# reference pages are available to developers wanting a list of
-# everything.)
-
-INPUT = @srcdir@/../defs.h \
- @srcdir@/../gdbtypes.h \
- @srcdir@/../minsyms.h \
- @srcdir@/../utils.h
-
-HTML_OUTPUT = ./doxy/gdb-api
-
-# Suppress classes/structs local to source files, they are unlikely
-# to be of general API interest.
-
-EXTRACT_LOCAL_CLASSES = NO
diff --git a/gdb/doc/Doxyfile-gdb-xref.in b/gdb/doc/Doxyfile-gdb-xref.in
deleted file mode 100644
index a93ac0397a9..00000000000
--- a/gdb/doc/Doxyfile-gdb-xref.in
+++ /dev/null
@@ -1,44 +0,0 @@
-# Copyright (C) 2014-2026 Free Software Foundation, Inc.
-
-# Doxygen file for a full GDB cross-reference.
-# This file is part of GDB.
-
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-@INCLUDE = Doxyfile-base
-
-PROJECT_NAME = "GDB (xrefs)"
-
-# Get all the sources.
-
-INPUT = @srcdir@/../ ../
-
-HTML_OUTPUT = ./doxy/gdb-xref
-
-# We want to use the XML for analysis and statistics.
-
-GENERATE_XML = YES
-XML_OUTPUT = ./doxy/xml
-
-# Include everything possible.
-
-EXTRACT_PRIVATE = YES
-EXTRACT_STATIC = YES
-
-# Build a full cross-reference.
-
-REFERENCED_BY_RELATION = YES
-REFERENCES_RELATION = YES
-REFERENCES_LINK_SOURCE = NO
-
diff --git a/gdb/doc/Doxyfile-gdbserver.in b/gdb/doc/Doxyfile-gdbserver.in
deleted file mode 100644
index 82723b1d3c2..00000000000
--- a/gdb/doc/Doxyfile-gdbserver.in
+++ /dev/null
@@ -1,45 +0,0 @@
-# Copyright (C) 2014-2026 Free Software Foundation, Inc.
-
-# Doxygen file for a GDBserver cross-reference.
-# This file is part of GDB.
-
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-@INCLUDE = Doxyfile-base
-
-PROJECT_NAME = "GDBserver"
-
-# Override the general include path.
-
-INCLUDE_PATH = @srcdir@/../common @srcdir@/../../include/
-
-# Include just the directories that go into GDBserver.
-
-INPUT = @srcdir@/../gdbserver @srcdir@/../common @srcdir@/../nat
-
-# Override the generic exclude list.
-# gdbreplay is its own program, avoid duplicate symbols.
-
-EXCLUDE = @srcdir@/../gdbserver/gdbreplay.c
-
-HTML_OUTPUT = ./doxy/gdbserver
-
-# Build a full cross-reference.
-
-REFERENCED_BY_RELATION = YES
-REFERENCES_RELATION = YES
-REFERENCES_LINK_SOURCE = NO
-
-EXTRACT_PRIVATE = YES
-EXTRACT_STATIC = YES
diff --git a/gdb/doc/Makefile.in b/gdb/doc/Makefile.in
index 09afc86de2f..cc0676dd288 100644
--- a/gdb/doc/Makefile.in
+++ b/gdb/doc/Makefile.in
@@ -203,42 +203,6 @@ html: $(HTMLFILES)
pdf: $(PDFFILES)
man: $(MANS)
-DOXYGEN = doxygen
-doxyedit = sed -e 's,@srcdir\@,$(srcdir),g'
-
-doxy: doxy/index.html \
- doxy/gdb-api/index.html \
- doxy/gdb-xref/index.html \
- doxy/gdbserver/index.html
-
-doxy/index.html: $(srcdir)/doxy-index.in
- -mkdir -p doxy
- cp $(srcdir)/doxy-index.in doxy/index.html
-
-doxy/gdb-api/index.html: Doxyfile-gdb-api Doxyfile-base
- -mkdir -p doxy
- $(DOXYGEN) Doxyfile-gdb-api
-
-doxy/gdb-xref/index.html: Doxyfile-gdb-xref Doxyfile-base
- -mkdir -p doxy
- $(DOXYGEN) Doxyfile-gdb-xref
-
-doxy/gdbserver/index.html: Doxyfile-gdbserver Doxyfile-base
- -mkdir -p doxy
- $(DOXYGEN) Doxyfile-gdbserver
-
-Doxyfile-base: $(srcdir)/Doxyfile-base.in
- $(doxyedit) $(srcdir)/Doxyfile-base.in >Doxyfile-base
-
-Doxyfile-gdb-api: $(srcdir)/Doxyfile-gdb-api.in
- $(doxyedit) $(srcdir)/Doxyfile-gdb-api.in >Doxyfile-gdb-api
-
-Doxyfile-gdb-xref: $(srcdir)/Doxyfile-gdb-xref.in
- $(doxyedit) $(srcdir)/Doxyfile-gdb-xref.in >Doxyfile-gdb-xref
-
-Doxyfile-gdbserver: $(srcdir)/Doxyfile-gdbserver.in
- $(doxyedit) $(srcdir)/Doxyfile-gdbserver.in >Doxyfile-gdbserver
-
all-doc: info man dvi ps pdf html
diststuff: info man
diff --git a/gdb/doc/doxy-index.in b/gdb/doc/doxy-index.in
deleted file mode 100644
index e386836462e..00000000000
--- a/gdb/doc/doxy-index.in
+++ /dev/null
@@ -1,76 +0,0 @@
-<!-- Copyright (C) 2014-2026 Free Software Foundation, Inc.
-
-Navigation page for Doxygen-generated GDB info.
-
-This file is part of GDB.
-
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 3 of the License, or
-(at your option) any later version.
-
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-GNU General Public License for more details.
-
-You should have received a copy of the GNU General Public License
-along with this program. If not, see <http://www.gnu.org/licenses/>.
--->
-
-<html lang="en">
-<head>
-<title>GDB Internals (Doxygen)</title>
-<meta http-equiv="Content-Type" content="text/html">
-<meta name="description" content="GDB Internals (Doxygen)">
-<meta http-equiv="Content-Style-Type" content="text/css">
-<style type="text/css">
-</style>
-</head>
-<body>
-
-<p>
-This page is the index to doxygen-generated pages describing GDB
-internals.
-
-<p>
-These are primarily useful for people working on GDB itself. Users of
-GDB, people working with stubs, and people working on frontends to GDB
-are all better served by the GDB manual, which is the official
-definition for all of GDB's external interfaces.
-
-<h2>
-<a href="gdb-api/index.html">GDB internal API</a>
-</h2>
-
-<p>
-The GDB internal API consists of all definitions and functions
-that are of interest to multiple areas of the debugger.
-
-<p>
-Note that this internal API reference lists only header files for
-which doxygen-type documentation has been added, and explicitly listed
-as a part of the API.
-
-<h2>
-<a href="gdb-xref/index.html">GDB full source cross-reference</a>
-</h2>
-
-<p>
-This cross-reference includes all of the GDB sources proper, omitting
-support libraries such as BFD, and files that are not compiled into
-GDB.
-
-<p>
-It is not guaranteed to report all uses of a construct, and you should
-independently check non-use of a symbol before trying to remove it.
-
-<h2>
-<a href="gdbserver/index.html">GDBserver source cross-reference</a>
-</h2>
-
-<p>
-Similar to the GDB cross-reference, but for GDBserver sources.
-
-</body>
-</html>
diff --git a/gdb/doc/filter-for-doxygen b/gdb/doc/filter-for-doxygen
deleted file mode 100755
index 457d088c104..00000000000
--- a/gdb/doc/filter-for-doxygen
+++ /dev/null
@@ -1,14 +0,0 @@
-#!/bin/sh
-
-# This filters GDB source before Doxygen can get confused by it;
-# this script is listed in the doxyfile. The output is not very
-# pretty, but at least we get output that Doxygen can understand.
-#
-# $1 is a source file of some kind. The source we wish doxygen to
-# process is put on stdout.
-
-# (Adapted from gcc/contrib/filter_gcc_for_doxygen)
-
-dir=`dirname $0`
-perl $dir/filter-params.pl < $1
-exit 0
diff --git a/gdb/doc/filter-params.pl b/gdb/doc/filter-params.pl
deleted file mode 100644
index b5d0f6b53ef..00000000000
--- a/gdb/doc/filter-params.pl
+++ /dev/null
@@ -1,11 +0,0 @@
-#!/usr/bin/perl
-
-# This Perl script tweaks GDB sources to be more useful for Doxygen.
-
-while (<>) {
- # Allow "/* * " as an equivalent to "/** ", better for Emacs compat.
- s/\/\* \* /\/** /sg;
- # Manually expand macro seen in structs and such.
- s/ENUM_BITFIELD[ \t]*\((.*?)\)/__extension__ enum $1/sg;
- print;
-}
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 2/4] [gdb] Remove '/* *' multi-line comment marker
2026-06-01 10:53 [PATCH v2 0/4] [gdb] Remove doxygen support Tom de Vries
2026-06-01 10:53 ` [PATCH v2 1/4] [gdb/doc] " Tom de Vries
@ 2026-06-01 10:53 ` Tom de Vries
2026-06-01 10:53 ` [PATCH v2 3/4] [gdb] Remove \page marker Tom de Vries
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Tom de Vries @ 2026-06-01 10:53 UTC (permalink / raw)
To: gdb-patches
Doxygen supports a multi-line comment marker '/**'. In GDB we're using
something slightly similar: '/* *' [1].
Drop this and just use '/*'.
Result of:
...
$ find gdb* -type f -name "*.[ch]" -o -name "*.def" \
| egrep -v /testsuite/ \
| xargs sed -i 's%/\* \* %/* %'
...
and manually reverting the change in the comment for BINOP_MUL in
gdb/std-operator.def.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34186
Approved-By: Kevin Buettner <kevinb@redhat.com>
[1] https://sourceware.org/gdb/wiki/DoxygenForGDB
---
gdb/cli/cli-script.h | 6 +-
gdb/defs.h | 70 ++++----
gdb/dwarf2/call-site.h | 30 ++--
gdb/gdbtypes.h | 326 +++++++++++++++++++-------------------
gdb/main.h | 2 +-
gdb/type-codes.def | 24 +--
gdb/valprint.h | 2 +-
gdbsupport/common-types.h | 12 +-
gdbsupport/host-defs.h | 4 +-
9 files changed, 238 insertions(+), 238 deletions(-)
diff --git a/gdb/cli/cli-script.h b/gdb/cli/cli-script.h
index 7b486113b8b..4937ba0456e 100644
--- a/gdb/cli/cli-script.h
+++ b/gdb/cli/cli-script.h
@@ -23,7 +23,7 @@
struct ui_file;
struct cmd_list_element;
-/* * Control types for commands. */
+/* Control types for commands. */
enum misc_command_type
{
@@ -70,7 +70,7 @@ typedef std::shared_ptr<command_line> counted_command_line;
/* A unique_ptr specialization for command_line. */
typedef std::unique_ptr<command_line, command_lines_deleter> command_line_up;
-/* * Structure for saved commands lines (for breakpoints, defined
+/* Structure for saved commands lines (for breakpoints, defined
commands, etc). */
struct command_line
@@ -97,7 +97,7 @@ struct command_line
compile;
}
control_u;
- /* * For composite commands, the nested lists of commands. For
+ /* For composite commands, the nested lists of commands. For
example, for "if" command this will contain the then branch and
the else branch, if that is available. */
counted_command_line body_list_0;
diff --git a/gdb/defs.h b/gdb/defs.h
index ea3d31a2426..5f123184d65 100644
--- a/gdb/defs.h
+++ b/gdb/defs.h
@@ -81,20 +81,20 @@
#define O_BINARY 0
#endif
-/* * System root path, used to find libraries etc. */
+/* System root path, used to find libraries etc. */
extern std::string gdb_sysroot;
-/* * GDB datadir, used to store data files. */
+/* GDB datadir, used to store data files. */
extern std::string gdb_datadir;
-/* * If not empty, the possibly relocated path to python's "lib" directory
+/* If not empty, the possibly relocated path to python's "lib" directory
specified with --with-python. */
extern std::string python_libdir;
-/* * Search path for separate debug files. */
+/* Search path for separate debug files. */
extern std::string debug_file_directory;
-/* * Languages represented in the symbol table and elsewhere.
+/* Languages represented in the symbol table and elsewhere.
This should probably be in language.h, but since enum's can't
be forward declared to satisfy opaque references before their
actual definition, needs to be here.
@@ -136,7 +136,7 @@ static_assert (nr_languages <= (1 << LANGUAGE_BITS));
/* The number of bytes needed to represent all languages. */
#define LANGUAGE_BYTES ((LANGUAGE_BITS + HOST_CHAR_BIT - 1) / HOST_CHAR_BIT)
-/* * A generic, not quite boolean, enumeration. This is used for
+/* A generic, not quite boolean, enumeration. This is used for
set/show commands in which the options are on/off/automatic. */
enum auto_boolean
{
@@ -145,28 +145,28 @@ enum auto_boolean
AUTO_BOOLEAN_AUTO
};
-/* * Potential ways that a function can return a value of a given
+/* Potential ways that a function can return a value of a given
type. */
enum return_value_convention
{
- /* * Where the return value has been squeezed into one or more
+ /* Where the return value has been squeezed into one or more
registers. */
RETURN_VALUE_REGISTER_CONVENTION,
- /* * Commonly known as the "struct return convention". The caller
+ /* Commonly known as the "struct return convention". The caller
passes an additional hidden first parameter to the caller. That
parameter contains the address at which the value being returned
should be stored. While typically, and historically, used for
large structs, this is convention is applied to values of many
different types. */
RETURN_VALUE_STRUCT_CONVENTION,
- /* * Like the "struct return convention" above, but where the ABI
+ /* Like the "struct return convention" above, but where the ABI
guarantees that the called function stores the address at which
the value being returned is stored in a well-defined location,
such as a register or memory slot in the stack frame. Don't use
this if the ABI doesn't explicitly guarantees this. */
RETURN_VALUE_ABI_RETURNS_ADDRESS,
- /* * Like the "struct return convention" above, but where the ABI
+ /* Like the "struct return convention" above, but where the ABI
guarantees that the address at which the value being returned is
stored will be available in a well-defined location, such as a
register or memory slot in the stack frame. Don't use this if
@@ -212,68 +212,68 @@ extern int print_address_symbolic (struct gdbarch *, CORE_ADDR,
extern void print_address (struct gdbarch *, CORE_ADDR, struct ui_file *);
extern const char *pc_prefix (CORE_ADDR);
-/* * Possible lvalue types. Like enum language, this should be in
+/* Possible lvalue types. Like enum language, this should be in
value.h, but needs to be here for the same reason. */
enum lval_type
{
- /* * Not an lval. */
+ /* Not an lval. */
not_lval,
- /* * In memory. */
+ /* In memory. */
lval_memory,
- /* * In a register. Registers are relative to a frame. */
+ /* In a register. Registers are relative to a frame. */
lval_register,
- /* * In a gdb internal variable. */
+ /* In a gdb internal variable. */
lval_internalvar,
- /* * Value encapsulates a callable defined in an extension language. */
+ /* Value encapsulates a callable defined in an extension language. */
lval_xcallable,
- /* * Part of a gdb internal variable (structure field). */
+ /* Part of a gdb internal variable (structure field). */
lval_internalvar_component,
- /* * Value's bits are fetched and stored using functions provided
+ /* Value's bits are fetched and stored using functions provided
by its creator. */
lval_computed
};
-/* * Parameters of the "info proc" command. */
+/* Parameters of the "info proc" command. */
enum info_proc_what
{
- /* * Display the default cmdline, cwd and exe outputs. */
+ /* Display the default cmdline, cwd and exe outputs. */
IP_MINIMAL,
- /* * Display `info proc mappings'. */
+ /* Display `info proc mappings'. */
IP_MAPPINGS,
- /* * Display `info proc status'. */
+ /* Display `info proc status'. */
IP_STATUS,
- /* * Display `info proc stat'. */
+ /* Display `info proc stat'. */
IP_STAT,
- /* * Display `info proc cmdline'. */
+ /* Display `info proc cmdline'. */
IP_CMDLINE,
- /* * Display `info proc environ'. */
+ /* Display `info proc environ'. */
IP_ENVIRON,
- /* * Display `info proc exe'. */
+ /* Display `info proc exe'. */
IP_EXE,
- /* * Display `info proc cwd'. */
+ /* Display `info proc cwd'. */
IP_CWD,
- /* * Display `info proc files'. */
+ /* Display `info proc files'. */
IP_FILES,
- /* * Display all of the above. */
+ /* Display all of the above. */
IP_ALL
};
-/* * Default radixes for input and output. Only some values supported. */
+/* Default radixes for input and output. Only some values supported. */
extern unsigned input_radix;
extern unsigned output_radix;
-/* * Optional native machine support. Non-native (and possibly pure
+/* Optional native machine support. Non-native (and possibly pure
multi-arch) targets do not need a "nm.h" file. This will be a
symlink to one of the nm-*.h files, built by the `configure'
script. */
@@ -294,7 +294,7 @@ extern unsigned output_radix;
# include "fopen-bin.h"
#endif
-/* * Convert a LONGEST to an int. This is used in contexts (e.g. number of
+/* Convert a LONGEST to an int. This is used in contexts (e.g. number of
arguments to a function, number in a value history, register number, etc.)
where the value must not be larger than can fit in an int. */
@@ -359,10 +359,10 @@ extern int (*deprecated_ui_load_progress_hook) (const char *section,
#define ISATTY(FP) (isatty (fileno (FP)))
#endif
-/* * A width that can achieve a better legibility for GDB MI mode. */
+/* A width that can achieve a better legibility for GDB MI mode. */
#define GDB_MI_MSG_WIDTH 80
-/* * Special block numbers */
+/* Special block numbers */
enum block_enum
{
diff --git a/gdb/dwarf2/call-site.h b/gdb/dwarf2/call-site.h
index 5e2e270e383..cfd2953cce1 100644
--- a/gdb/dwarf2/call-site.h
+++ b/gdb/dwarf2/call-site.h
@@ -35,13 +35,13 @@ struct dwarf2_per_objfile;
enum call_site_parameter_kind
{
- /* * Use field call_site_parameter.u.dwarf_reg. */
+ /* Use field call_site_parameter.u.dwarf_reg. */
CALL_SITE_PARAMETER_DWARF_REG,
- /* * Use field call_site_parameter.u.fb_offset. */
+ /* Use field call_site_parameter.u.fb_offset. */
CALL_SITE_PARAMETER_FB_OFFSET,
- /* * Use field call_site_parameter.u.param_offset. */
+ /* Use field call_site_parameter.u.param_offset. */
CALL_SITE_PARAMETER_PARAM_OFFSET
};
@@ -116,25 +116,25 @@ struct call_site_target
} addresses;
} m_loc;
- /* * Discriminant for union field_location. */
+ /* Discriminant for union field_location. */
enum kind m_loc_kind;
};
union call_site_parameter_u
{
- /* * DW_TAG_formal_parameter's DW_AT_location's DW_OP_regX
+ /* DW_TAG_formal_parameter's DW_AT_location's DW_OP_regX
as DWARF register number, for register passed
parameters. */
int dwarf_reg;
- /* * Offset from the callee's frame base, for stack passed
+ /* Offset from the callee's frame base, for stack passed
parameters. This equals offset from the caller's stack
pointer. */
CORE_ADDR fb_offset;
- /* * Offset relative to the start of this PER_CU to
+ /* Offset relative to the start of this PER_CU to
DW_TAG_formal_parameter which is referenced by both
caller and the callee. */
@@ -156,19 +156,19 @@ struct call_site_parameter
union call_site_parameter_u u;
- /* * DW_TAG_formal_parameter's DW_AT_call_value. It is never NULL. */
+ /* DW_TAG_formal_parameter's DW_AT_call_value. It is never NULL. */
const gdb_byte *value;
size_t value_size;
- /* * DW_TAG_formal_parameter's DW_AT_call_data_value.
+ /* DW_TAG_formal_parameter's DW_AT_call_data_value.
It may be NULL if not provided by DWARF. */
const gdb_byte *data_value;
size_t data_value_size;
};
-/* * A place where a function gets called from, represented by
+/* A place where a function gets called from, represented by
DW_TAG_call_site. It can be looked up from symtab->call_site_htab. */
struct call_site
@@ -202,20 +202,20 @@ struct call_site
caller_frame, callback);
}
- /* * List successor with head in FUNC_TYPE.TAIL_CALL_LIST. */
+ /* List successor with head in FUNC_TYPE.TAIL_CALL_LIST. */
struct call_site *tail_call_next = nullptr;
- /* * Describe DW_AT_call_target. Missing attribute uses
+ /* Describe DW_AT_call_target. Missing attribute uses
m_loc_kind == DWARF_BLOCK with m_loc.dwarf_block == nullptr. */
struct call_site_target target {};
- /* * Size of the PARAMETER array. */
+ /* Size of the PARAMETER array. */
unsigned parameter_count = 0;
- /* * CU of the function where the call is located. It gets used
+ /* CU of the function where the call is located. It gets used
for DWARF blocks execution in the parameter array below. */
dwarf2_per_cu *const per_cu = nullptr;
@@ -229,7 +229,7 @@ struct call_site
const unrelocated_addr m_unrelocated_pc;
public:
- /* * Describe DW_TAG_call_site's DW_TAG_formal_parameter. */
+ /* Describe DW_TAG_call_site's DW_TAG_formal_parameter. */
struct call_site_parameter parameter[];
};
diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h
index 027b814fe6a..dc3703d74f9 100644
--- a/gdb/gdbtypes.h
+++ b/gdb/gdbtypes.h
@@ -22,7 +22,7 @@
#ifndef GDB_GDBTYPES_H
#define GDB_GDBTYPES_H
-/* * \page gdbtypes GDB Types
+/* \page gdbtypes GDB Types
GDB represents all the different kinds of types in programming
languages using a common representation defined in gdbtypes.h.
@@ -60,7 +60,7 @@ struct dwarf2_per_cu;
struct dwarf2_per_objfile;
struct dwarf2_property_baton;
-/* * Different kinds of data types are distinguished by the `code'
+/* Different kinds of data types are distinguished by the `code'
field. */
enum type_code
@@ -73,7 +73,7 @@ enum type_code
};
-/* * Some bits for the type's instance_flags word. See the macros
+/* Some bits for the type's instance_flags word. See the macros
below for documentation on each bit. */
enum type_instance_flag_value : unsigned
@@ -91,53 +91,53 @@ enum type_instance_flag_value : unsigned
DEF_ENUM_FLAGS_TYPE (enum type_instance_flag_value, type_instance_flags);
-/* * Not textual. By default, GDB treats all single byte integers as
+/* Not textual. By default, GDB treats all single byte integers as
characters (or elements of strings) unless this flag is set. */
#define TYPE_NOTTEXT(t) (((t)->instance_flags ()) & TYPE_INSTANCE_FLAG_NOTTEXT)
-/* * Constant type. If this is set, the corresponding type has a
+/* Constant type. If this is set, the corresponding type has a
const modifier. */
#define TYPE_CONST(t) ((((t)->instance_flags ()) & TYPE_INSTANCE_FLAG_CONST) != 0)
-/* * Volatile type. If this is set, the corresponding type has a
+/* Volatile type. If this is set, the corresponding type has a
volatile modifier. */
#define TYPE_VOLATILE(t) \
((((t)->instance_flags ()) & TYPE_INSTANCE_FLAG_VOLATILE) != 0)
-/* * Restrict type. If this is set, the corresponding type has a
+/* Restrict type. If this is set, the corresponding type has a
restrict modifier. */
#define TYPE_RESTRICT(t) \
((((t)->instance_flags ()) & TYPE_INSTANCE_FLAG_RESTRICT) != 0)
-/* * Atomic type. If this is set, the corresponding type has an
+/* Atomic type. If this is set, the corresponding type has an
_Atomic modifier. */
#define TYPE_ATOMIC(t) \
((((t)->instance_flags ()) & TYPE_INSTANCE_FLAG_ATOMIC) != 0)
-/* * True if this type represents either an lvalue or lvalue reference type. */
+/* True if this type represents either an lvalue or lvalue reference type. */
#define TYPE_IS_REFERENCE(t) \
((t)->code () == TYPE_CODE_REF || (t)->code () == TYPE_CODE_RVALUE_REF)
-/* * True if this type is allocatable. */
+/* True if this type is allocatable. */
#define TYPE_IS_ALLOCATABLE(t) \
((t)->dyn_prop (DYN_PROP_ALLOCATED) != NULL)
-/* * True if this type has variant parts. */
+/* True if this type has variant parts. */
#define TYPE_HAS_VARIANT_PARTS(t) \
((t)->dyn_prop (DYN_PROP_VARIANT_PARTS) != nullptr)
-/* * True if this type has a dynamic length. */
+/* True if this type has a dynamic length. */
#define TYPE_HAS_DYNAMIC_LENGTH(t) \
(((t)->dyn_prop (DYN_PROP_BYTE_SIZE) != nullptr) \
|| ((t)->dyn_prop (DYN_PROP_BIT_SIZE) != nullptr))
-/* * Instruction-space delimited type. This is for Harvard architectures
+/* Instruction-space delimited type. This is for Harvard architectures
which have separate instruction and data address spaces (and perhaps
others).
@@ -162,7 +162,7 @@ DEF_ENUM_FLAGS_TYPE (enum type_instance_flag_value, type_instance_flags);
#define TYPE_DATA_SPACE(t) \
((((t)->instance_flags ()) & TYPE_INSTANCE_FLAG_DATA_SPACE) != 0)
-/* * Address class flags. Some environments provide for pointers
+/* Address class flags. Some environments provide for pointers
whose size is different from that of a normal pointer or address
types where the bits are interpreted differently than normal
addresses. The TYPE_INSTANCE_FLAG_ADDRESS_CLASS_n flags may be used in
@@ -178,15 +178,15 @@ DEF_ENUM_FLAGS_TYPE (enum type_instance_flag_value, type_instance_flags);
#define TYPE_ADDRESS_CLASS_ALL(t) (((t)->instance_flags ()) \
& TYPE_INSTANCE_FLAG_ADDRESS_CLASS_ALL)
-/* * Information about a single discriminant. */
+/* Information about a single discriminant. */
struct discriminant_range
{
- /* * The range of values for the variant. This is an inclusive
+ /* The range of values for the variant. This is an inclusive
range. */
ULONGEST low, high;
- /* * Return true if VALUE is contained in this range. IS_UNSIGNED
+ /* Return true if VALUE is contained in this range. IS_UNSIGNED
is true if this should be an unsigned comparison; false for
signed. */
bool contains (ULONGEST value, bool is_unsigned) const
@@ -200,7 +200,7 @@ struct discriminant_range
struct variant_part;
-/* * A single variant. A variant has a list of discriminant values.
+/* A single variant. A variant has a list of discriminant values.
When the discriminator matches one of these, the variant is
enabled. Each variant controls zero or more fields; and may also
control other variant parts as well. This struct corresponds to
@@ -208,20 +208,20 @@ struct variant_part;
struct variant : allocate_on_obstack<variant>
{
- /* * The discriminant ranges for this variant. */
+ /* The discriminant ranges for this variant. */
gdb::array_view<discriminant_range> discriminants;
- /* * The fields controlled by this variant. This is inclusive on
+ /* The fields controlled by this variant. This is inclusive on
the low end and exclusive on the high end. A variant may not
control any fields, in which case the two values will be equal.
These are indexes into the type's array of fields. */
int first_field;
int last_field;
- /* * Variant parts controlled by this variant. */
+ /* Variant parts controlled by this variant. */
gdb::array_view<variant_part> parts;
- /* * Return true if this is the default variant. The default
+ /* Return true if this is the default variant. The default
variant can be recognized because it has no associated
discriminants. */
bool is_default () const
@@ -229,28 +229,28 @@ struct variant : allocate_on_obstack<variant>
return discriminants.empty ();
}
- /* * Return true if this variant matches VALUE. IS_UNSIGNED is true
+ /* Return true if this variant matches VALUE. IS_UNSIGNED is true
if this should be an unsigned comparison; false for signed. */
bool matches (ULONGEST value, bool is_unsigned) const;
};
-/* * A variant part. Each variant part has an optional discriminant
+/* A variant part. Each variant part has an optional discriminant
and holds an array of variants. This struct corresponds to
DW_TAG_variant_part in DWARF. */
struct variant_part : allocate_on_obstack<variant_part>
{
- /* * The index of the discriminant field in the outer type. This is
+ /* The index of the discriminant field in the outer type. This is
an index into the type's array of fields. If this is -1, there
is no discriminant, and only the default variant can be
considered to be selected. */
int discriminant_index;
- /* * True if this discriminant is unsigned; false if signed. This
+ /* True if this discriminant is unsigned; false if signed. This
comes from the type of the discriminant. */
bool is_unsigned;
- /* * The variants that are controlled by this variant part. Note
+ /* The variants that are controlled by this variant part. Note
that these will always be sorted by field number. */
gdb::array_view<variant> variants;
};
@@ -300,7 +300,7 @@ union dynamic_prop_data
const char *variable_name;
};
-/* * Used to store a dynamic property. */
+/* Used to store a dynamic property. */
struct dynamic_prop
{
@@ -430,7 +430,7 @@ static inline bool operator!= (const dynamic_prop &l, const dynamic_prop &r)
return !(l == r);
}
-/* * Define a type's dynamic property node kind. */
+/* Define a type's dynamic property node kind. */
enum dynamic_prop_node_kind
{
/* A property providing a type's data location.
@@ -467,7 +467,7 @@ enum dynamic_prop_node_kind
DYN_PROP_BIT_SIZE,
};
-/* * List for dynamic type attributes. */
+/* List for dynamic type attributes. */
struct dynamic_prop_list
{
/* The kind of dynamic prop in this node. */
@@ -480,7 +480,7 @@ struct dynamic_prop_list
struct dynamic_prop_list *next;
};
-/* * Determine which field of the union main_type.fields[x].loc is
+/* Determine which field of the union main_type.fields[x].loc is
used. */
enum field_loc_kind
@@ -495,7 +495,7 @@ enum field_loc_kind
FIELD_LOC_KIND_DWARF_BLOCK_BITPOS,
};
-/* * A discriminant to determine which field in the
+/* A discriminant to determine which field in the
main_type.type_specific union is being used, if any.
For types such as TYPE_CODE_FLT, the use of this
@@ -527,17 +527,17 @@ union type_owner
union field_location
{
- /* * Position of this field, counting in bits from start of
+ /* Position of this field, counting in bits from start of
containing structure. For big-endian targets, it is the bit
offset to the MSB. For little-endian targets, it is the bit
offset to the LSB. */
LONGEST bitpos;
- /* * Enum value. */
+ /* Enum value. */
LONGEST enumval;
- /* * For a static field, if TYPE_FIELD_STATIC_HAS_ADDR then
+ /* For a static field, if TYPE_FIELD_STATIC_HAS_ADDR then
physaddr is the location (in the target) of the static
field. Otherwise, physname is the mangled label of the
static field. */
@@ -545,7 +545,7 @@ union field_location
CORE_ADDR physaddr;
const char *physname;
- /* * The field location can be computed by evaluating the
+ /* The field location can be computed by evaluating the
following DWARF block. Its DATA is allocated on
objfile_obstack - no CU load is needed to access it. */
@@ -742,7 +742,7 @@ struct field
union field_location m_loc;
- /* * For a function or member type, this is 1 if the argument is
+ /* For a function or member type, this is 1 if the argument is
marked artificial. Artificial arguments should not be shown
to the user. For TYPE_CODE_RANGE it is set if the specific
bound is not defined. */
@@ -754,14 +754,14 @@ struct field
/* Whether the field is 'ignored'. */
bool m_ignored : 1;
- /* * Discriminant for union field_location. */
+ /* Discriminant for union field_location. */
field_loc_kind m_loc_kind : 3;
/* Accessibility of the field. */
enum accessibility m_accessibility;
- /* * Size of this field, in bits, or zero if not packed.
+ /* Size of this field, in bits, or zero if not packed.
If non-zero in an array type, indicates the element size in
bits (used only in Ada at the moment).
For an unpacked field, the field's type's length
@@ -769,13 +769,13 @@ struct field
unsigned int m_bitsize;
- /* * In a struct or union type, type of this field.
+ /* In a struct or union type, type of this field.
- In a function or member type, type of this argument.
- In an array type, the domain-type of the array. */
struct type *m_type;
- /* * Name of field, value or argument.
+ /* Name of field, value or argument.
NULL for range bounds, array domains, and member function
arguments. */
@@ -799,11 +799,11 @@ struct range_bounds
|| high.kind () == PROP_OPTIMIZED_OUT);
}
- /* * Low bound of range. */
+ /* Low bound of range. */
struct dynamic_prop low;
- /* * High bound of range. */
+ /* High bound of range. */
struct dynamic_prop high;
@@ -814,7 +814,7 @@ struct range_bounds
struct dynamic_prop stride;
- /* * The bias. Sometimes a range value is biased before storage.
+ /* The bias. Sometimes a range value is biased before storage.
The bias is added to the stored bits to form the true value. */
LONGEST bias;
@@ -846,65 +846,65 @@ static inline bool operator!= (const range_bounds &l, const range_bounds &r)
union type_specific
{
- /* * CPLUS_STUFF is for TYPE_CODE_STRUCT. It is initialized to
+ /* CPLUS_STUFF is for TYPE_CODE_STRUCT. It is initialized to
point to cplus_struct_default, a default static instance of a
struct cplus_struct_type. */
struct cplus_struct_type *cplus_stuff;
- /* * GNAT_STUFF is for types for which the GNAT Ada compiler
+ /* GNAT_STUFF is for types for which the GNAT Ada compiler
provides additional information. */
struct gnat_aux_type *gnat_stuff;
- /* * FLOATFORMAT is for TYPE_CODE_FLT. It is a pointer to a
+ /* FLOATFORMAT is for TYPE_CODE_FLT. It is a pointer to a
floatformat object that describes the floating-point value
that resides within the type. */
const struct floatformat *floatformat;
- /* * For TYPE_CODE_FUNC and TYPE_CODE_METHOD types. */
+ /* For TYPE_CODE_FUNC and TYPE_CODE_METHOD types. */
struct func_type *func_stuff;
- /* * For types that are pointer to member types (TYPE_CODE_METHODPTR,
+ /* For types that are pointer to member types (TYPE_CODE_METHODPTR,
TYPE_CODE_MEMBERPTR), SELF_TYPE is the type that this pointer
is a member of. */
struct type *self_type;
- /* * For TYPE_CODE_FIXED_POINT types, the info necessary to decode
+ /* For TYPE_CODE_FIXED_POINT types, the info necessary to decode
values of that type. */
struct fixed_point_type_info *fixed_point_info;
- /* * An integer-like scalar type may be stored in just part of its
+ /* An integer-like scalar type may be stored in just part of its
enclosing storage bytes. This structure describes this
situation. */
struct
{
- /* * The bit size of the integer. This can be 0. For integers
+ /* The bit size of the integer. This can be 0. For integers
that fill their storage (the ordinary case), this field holds
the byte size times 8. */
unsigned short bit_size;
- /* * The bit offset of the integer. This is ordinarily 0, and can
+ /* The bit offset of the integer. This is ordinarily 0, and can
only be non-zero if the bit size is less than the storage
size. */
unsigned short bit_offset;
} int_stuff;
};
-/* * Main structure representing a type in GDB.
+/* Main structure representing a type in GDB.
This structure is space-critical. Its layout has been tweaked to
reduce the space used. */
struct main_type
{
- /* * Code for kind of type. */
+ /* Code for kind of type. */
type_code code : 8;
- /* * Flags about this type. These fields appear at this location
+ /* Flags about this type. These fields appear at this location
because they packs nicely here. See the TYPE_* macros for
documentation about these fields. */
@@ -920,17 +920,17 @@ struct main_type
unsigned int m_flag_objfile_owned : 1;
unsigned int m_flag_endianity_not_default : 1;
- /* * True if this type was declared with "class" rather than
+ /* True if this type was declared with "class" rather than
"struct". */
unsigned int m_flag_declared_class : 1;
- /* * True if this is an enum type with disjoint values. This
+ /* True if this is an enum type with disjoint values. This
affects how the enum is printed. */
unsigned int m_flag_flag_enum : 1;
- /* * For TYPE_CODE_ARRAY, this is true if this type is part of a
+ /* For TYPE_CODE_ARRAY, this is true if this type is part of a
multi-dimensional array. Multi-dimensional arrays are
represented internally as arrays of arrays, and this flag lets
gdb distinguish between multiple dimensions and an ordinary array
@@ -939,7 +939,7 @@ struct main_type
unsigned int m_multi_dimensional : 1;
- /* * A discriminant telling us which field of the type_specific
+ /* A discriminant telling us which field of the type_specific
union is being used for this type, if any. */
type_specific_kind type_specific_field : 3;
@@ -948,12 +948,12 @@ struct main_type
language m_lang : LANGUAGE_BITS;
- /* * Number of fields described for this type. This field appears
+ /* Number of fields described for this type. This field appears
at this location because it packs nicely here. */
unsigned int m_nfields;
- /* * Name of this type, or NULL if none.
+ /* Name of this type, or NULL if none.
This is used for printing only. For looking up a name, look for
a symbol in the VAR_DOMAIN. This is generally allocated in the
@@ -961,7 +961,7 @@ struct main_type
const char *name;
- /* * Every type is now associated with a particular objfile, and the
+ /* Every type is now associated with a particular objfile, and the
type is allocated on the objfile_obstack for that objfile. One
problem however, is that there are times when gdb allocates new
types while it is not in the process of reading symbols from a
@@ -975,7 +975,7 @@ struct main_type
union type_owner m_owner;
- /* * For a pointer type, describes the type of object pointed to.
+ /* For a pointer type, describes the type of object pointed to.
- For an array type, describes the type of the elements.
- For a function or method type, describes the type of the return value.
- For a range type, describes the type of the full range.
@@ -987,7 +987,7 @@ struct main_type
struct type *m_target_type;
- /* * For structure and union types, a description of each field.
+ /* For structure and union types, a description of each field.
For set and pascal array types, there is one "field",
whose type is the domain type of the set or array.
For range types, there are two "fields",
@@ -1007,7 +1007,7 @@ struct main_type
{
struct field *fields;
- /* * Union member used for range types. */
+ /* Union member used for range types. */
struct range_bounds *bounds;
@@ -1017,20 +1017,20 @@ struct main_type
} flds_bnds;
- /* * Slot to point to additional language-specific fields of this
+ /* Slot to point to additional language-specific fields of this
type. */
union type_specific type_specific;
- /* * Contains all dynamic type properties. */
+ /* Contains all dynamic type properties. */
struct dynamic_prop_list *dyn_prop_list;
};
-/* * Number of bits allocated for alignment. */
+/* Number of bits allocated for alignment. */
#define TYPE_ALIGN_BITS 8
-/* * A ``struct type'' describes a particular instance of a type, with
+/* A ``struct type'' describes a particular instance of a type, with
some particular qualification. */
struct type
@@ -1396,7 +1396,7 @@ struct type
this->main_type->m_multi_dimensional = value;
}
- /* * Assuming that THIS is a TYPE_CODE_FIXED_POINT, return a reference
+ /* Assuming that THIS is a TYPE_CODE_FIXED_POINT, return a reference
to this type's fixed_point_info. */
struct fixed_point_type_info &fixed_point_info () const
@@ -1407,7 +1407,7 @@ struct type
return *this->main_type->type_specific.fixed_point_info;
}
- /* * Assuming that THIS is a TYPE_CODE_FIXED_POINT, set this type's
+ /* Assuming that THIS is a TYPE_CODE_FIXED_POINT, set this type's
fixed_point_info to INFO. */
void set_fixed_point_info (struct fixed_point_type_info *info) const
@@ -1417,7 +1417,7 @@ struct type
this->main_type->type_specific.fixed_point_info = info;
}
- /* * Assuming that THIS is a TYPE_CODE_FIXED_POINT, return its base type.
+ /* Assuming that THIS is a TYPE_CODE_FIXED_POINT, return its base type.
In other words, this returns the type after having peeled all
intermediate type layers (such as TYPE_CODE_RANGE, for instance).
@@ -1426,22 +1426,22 @@ struct type
struct type *fixed_point_type_base_type ();
- /* * Assuming that THIS is a TYPE_CODE_FIXED_POINT, return its scaling
+ /* Assuming that THIS is a TYPE_CODE_FIXED_POINT, return its scaling
factor. */
const gdb_mpq &fixed_point_scaling_factor ();
- /* * Return the dynamic property of the requested KIND from this type's
+ /* Return the dynamic property of the requested KIND from this type's
list of dynamic properties. */
dynamic_prop *dyn_prop (dynamic_prop_node_kind kind) const;
- /* * Given a dynamic property PROP of a given KIND, add this dynamic
+ /* Given a dynamic property PROP of a given KIND, add this dynamic
property to this type.
This function assumes that this type is objfile-owned. */
void add_dyn_prop (dynamic_prop_node_kind kind, dynamic_prop prop);
- /* * Remove dynamic property of kind KIND from this type, if it exists. */
+ /* Remove dynamic property of kind KIND from this type, if it exists. */
void remove_dyn_prop (dynamic_prop_node_kind kind);
/* Return true if this type is owned by an objfile. Return false if it is
@@ -1498,7 +1498,7 @@ struct type
The return value is always non-nullptr. */
gdbarch *arch () const;
- /* * Return true if this is an integer type whose logical (bit) size
+ /* Return true if this is an integer type whose logical (bit) size
differs from its storage size; false otherwise. Always return
false for non-integer (i.e., non-TYPE_SPECIFIC_INT) types. */
bool bit_size_differs_p () const
@@ -1507,7 +1507,7 @@ struct type
&& main_type->type_specific.int_stuff.bit_size != 8 * length ());
}
- /* * Return the logical (bit) size for this integer type. Only
+ /* Return the logical (bit) size for this integer type. Only
valid for integer (TYPE_SPECIFIC_INT) types. */
unsigned short bit_size () const
{
@@ -1515,7 +1515,7 @@ struct type
return main_type->type_specific.int_stuff.bit_size;
}
- /* * Return the bit offset for this integer type. Only valid for
+ /* Return the bit offset for this integer type. Only valid for
integer (TYPE_SPECIFIC_INT) types. */
unsigned short bit_offset () const
{
@@ -1542,22 +1542,22 @@ struct type
enum language language () const
{ return main_type->m_lang; }
- /* * Type that is a pointer to this type.
+ /* Type that is a pointer to this type.
NULL if no such pointer-to type is known yet.
The debugger may add the address of such a type
if it has to construct one later. */
struct type *pointer_type;
- /* * C++: also need a reference type. */
+ /* C++: also need a reference type. */
struct type *reference_type;
- /* * A C++ rvalue reference type added in C++11. */
+ /* A C++ rvalue reference type added in C++11. */
struct type *rvalue_reference_type;
- /* * Variant chain. This points to a type that differs from this
+ /* Variant chain. This points to a type that differs from this
one only in qualifiers and length. Currently, the possible
qualifiers are const, volatile, code-space, data-space, and
address class. The length may differ only when one of the
@@ -1566,7 +1566,7 @@ struct type
struct type *chain;
- /* * The alignment for this type. Zero means that the alignment was
+ /* The alignment for this type. Zero means that the alignment was
not specified in the debug info. Note that this is stored in a
funny way: as the log base 2 (plus 1) of the alignment; so a
value of 1 means the alignment is 1, and a value of 9 means the
@@ -1574,7 +1574,7 @@ struct type
unsigned align_log2 : TYPE_ALIGN_BITS;
- /* * Flags specific to this instance of the type, indicating where
+ /* Flags specific to this instance of the type, indicating where
on the ring we are.
For TYPE_CODE_TYPEDEF the flags of the typedef type should be
@@ -1586,7 +1586,7 @@ struct type
check_typedef. */
unsigned m_instance_flags : 9;
- /* * Length of storage for a value of this type. The value is the
+ /* Length of storage for a value of this type. The value is the
expression in host bytes of what sizeof(type) would return. This
size includes padding. For example, an i386 extended-precision
floating point value really only occupies ten bytes, but most
@@ -1604,7 +1604,7 @@ struct type
ULONGEST m_length;
- /* * Core type, shared by a group of qualified types. */
+ /* Core type, shared by a group of qualified types. */
struct main_type *main_type;
};
@@ -1612,17 +1612,17 @@ struct type
struct fn_fieldlist
{
- /* * The overloaded name.
+ /* The overloaded name.
This is generally allocated in the objfile's obstack.
However stabsread.c sometimes uses malloc. */
const char *name;
- /* * The number of methods with this name. */
+ /* The number of methods with this name. */
int length;
- /* * The list of methods. */
+ /* The list of methods. */
struct fn_field *fn_fields;
};
@@ -1631,13 +1631,13 @@ struct fn_fieldlist
struct fn_field
{
- /* * This is the mangled name which we can look up to find the
+ /* This is the mangled name which we can look up to find the
address of the method (FIXME: it would be cleaner to have a
pointer to the struct symbol here instead). */
const char *physname;
- /* * The function type for the method.
+ /* The function type for the method.
(This comment used to say "The return value of the method", but
that's wrong. The function type is expected here, i.e. something
@@ -1645,7 +1645,7 @@ struct fn_field
struct type *type;
- /* * For virtual functions. First baseclass that defines this
+ /* For virtual functions. First baseclass that defines this
virtual function. */
struct type *fcontext;
@@ -1656,15 +1656,15 @@ struct fn_field
unsigned int is_volatile:1;
unsigned int is_artificial:1;
- /* * True if this function is a constructor, false otherwise. */
+ /* True if this function is a constructor, false otherwise. */
unsigned int is_constructor : 1;
- /* * True if this function is deleted, false otherwise. */
+ /* True if this function is deleted, false otherwise. */
unsigned int is_deleted : 1;
- /* * DW_AT_defaulted attribute for this function. The value is one
+ /* DW_AT_defaulted attribute for this function. The value is one
of the DW_DEFAULTED constants. */
dwarf_defaulted_attribute defaulted : 2;
@@ -1672,7 +1672,7 @@ struct fn_field
/* Accessibility of the field. */
enum accessibility accessibility;
- /* * Index into that baseclass's virtual function table, minus 2;
+ /* Index into that baseclass's virtual function table, minus 2;
else if static: VOFFSET_STATIC; else: 0. */
unsigned int voffset:16;
@@ -1683,12 +1683,12 @@ struct fn_field
struct decl_field
{
- /* * Unqualified name to be prefixed by owning class qualified
+ /* Unqualified name to be prefixed by owning class qualified
name. */
const char *name;
- /* * Type this typedef named NAME represents. */
+ /* Type this typedef named NAME represents. */
struct type *type;
@@ -1696,19 +1696,19 @@ struct decl_field
enum accessibility accessibility;
};
-/* * C++ language-specific information for TYPE_CODE_STRUCT and
+/* C++ language-specific information for TYPE_CODE_STRUCT and
TYPE_CODE_UNION nodes. */
struct cplus_struct_type
{
- /* * Number of base classes this type derives from. The
+ /* Number of base classes this type derives from. The
baseclasses are stored in the first N_BASECLASSES fields
(i.e. the `fields' field of the struct type). The only fields
of struct field that are used are: type, name, loc.bitpos. */
short n_baseclasses;
- /* * Field number of the virtual function table pointer in VPTR_BASETYPE.
+ /* Field number of the virtual function table pointer in VPTR_BASETYPE.
All access to this field must be through TYPE_VPTR_FIELDNO as one
thing it does is check whether the field has been initialized.
Initially TYPE_RAW_CPLUS_SPECIFIC has the value of cplus_struct_default,
@@ -1724,16 +1724,16 @@ struct cplus_struct_type
short vptr_fieldno;
- /* * Number of methods with unique names. All overloaded methods
+ /* Number of methods with unique names. All overloaded methods
with the same name count only once. */
short nfn_fields;
- /* * Number of template arguments. */
+ /* Number of template arguments. */
unsigned short n_template_arguments;
- /* * One if this struct is a dynamic class, as defined by the
+ /* One if this struct is a dynamic class, as defined by the
Itanium C++ ABI: if it requires a virtual table pointer,
because it or any of its base classes have one or more virtual
member functions or virtual base classes. Minus one if not
@@ -1741,17 +1741,17 @@ struct cplus_struct_type
int is_dynamic : 2;
- /* * The calling convention for this type, fetched from the
+ /* The calling convention for this type, fetched from the
DW_AT_calling_convention attribute. The value is one of the
DW_CC constants. */
dwarf_calling_convention calling_convention : 8;
- /* * The base class which defined the virtual function table pointer. */
+ /* The base class which defined the virtual function table pointer. */
struct type *vptr_basetype;
- /* * For classes, structures, and unions, a description of each
+ /* For classes, structures, and unions, a description of each
field, which consists of an overloaded name, followed by the
types of arguments that the method expects, and then the name
after it has been renamed to make it distinct.
@@ -1760,34 +1760,34 @@ struct cplus_struct_type
struct fn_fieldlist *fn_fieldlists;
- /* * typedefs defined inside this class. typedef_field points to
+ /* typedefs defined inside this class. typedef_field points to
an array of typedef_field_count elements. */
struct decl_field *typedef_field;
unsigned typedef_field_count;
- /* * The nested types defined by this type. nested_types points to
+ /* The nested types defined by this type. nested_types points to
an array of nested_types_count elements. */
struct decl_field *nested_types;
unsigned nested_types_count;
- /* * The template arguments. This is an array with
+ /* The template arguments. This is an array with
N_TEMPLATE_ARGUMENTS elements. This is NULL for non-template
classes. */
struct symbol **template_arguments;
};
-/* * Struct used to store conversion rankings. */
+/* Struct used to store conversion rankings. */
struct rank
{
short rank;
- /* * When two conversions are of the same type and therefore have
+ /* When two conversions are of the same type and therefore have
the same rank, subrank is used to differentiate the two.
Eg: Two derived-class-pointer to base-class-pointer conversions
@@ -1798,38 +1798,38 @@ struct rank
short subrank;
};
-/* * Used for ranking a function for overload resolution. */
+/* Used for ranking a function for overload resolution. */
typedef std::vector<rank> badness_vector;
-/* * GNAT Ada-specific information for various Ada types. */
+/* GNAT Ada-specific information for various Ada types. */
struct gnat_aux_type
{
- /* * Parallel type used to encode information about dynamic types
+ /* Parallel type used to encode information about dynamic types
used in Ada (such as variant records, variable-size array,
etc). */
struct type* descriptive_type;
};
-/* * For TYPE_CODE_FUNC and TYPE_CODE_METHOD types. */
+/* For TYPE_CODE_FUNC and TYPE_CODE_METHOD types. */
struct func_type
{
- /* * The calling convention for targets supporting multiple ABIs.
+ /* The calling convention for targets supporting multiple ABIs.
Right now this is only fetched from the Dwarf-2
DW_AT_calling_convention attribute. The value is one of the
DW_CC constants. */
dwarf_calling_convention calling_convention : 8;
- /* * Whether this function normally returns to its caller. It is
+ /* Whether this function normally returns to its caller. It is
set from the DW_AT_noreturn attribute if set on the
DW_TAG_subprogram. */
unsigned int is_noreturn : 1;
- /* * Only those DW_TAG_call_site's in this function that have
+ /* Only those DW_TAG_call_site's in this function that have
DW_AT_call_tail_call set are linked in this list. Function
without its tail call list complete
(DW_AT_call_all_tail_calls or its superset
@@ -1838,7 +1838,7 @@ struct func_type
struct call_site *tail_call_list;
- /* * For method types (TYPE_CODE_METHOD), the aggregate type that
+ /* For method types (TYPE_CODE_METHOD), the aggregate type that
contains the method. */
struct type *self_type;
@@ -1852,7 +1852,7 @@ struct fixed_point_type_info
gdb_mpq scaling_factor;
};
-/* * The default value of TYPE_CPLUS_SPECIFIC(T) points to this shared
+/* The default value of TYPE_CPLUS_SPECIFIC(T) points to this shared
static structure. */
extern const struct cplus_struct_type cplus_struct_default;
@@ -1878,12 +1878,12 @@ extern void allocate_gnat_aux_type (struct type *);
(TYPE_SPECIFIC_FIELD (type) = TYPE_SPECIFIC_GNAT_STUFF, \
TYPE_GNAT_SPECIFIC (type) = (struct gnat_aux_type *) &gnat_aux_default)
#define ALLOCATE_GNAT_AUX_TYPE(type) allocate_gnat_aux_type (type)
-/* * A macro that returns non-zero if the type-specific data should be
+/* A macro that returns non-zero if the type-specific data should be
read as "gnat-stuff". */
#define HAVE_GNAT_AUX_INFO(type) \
(TYPE_SPECIFIC_FIELD (type) == TYPE_SPECIFIC_GNAT_STUFF)
-/* * True if TYPE is known to be an Ada type of some kind. */
+/* True if TYPE is known to be an Ada type of some kind. */
#define ADA_TYPE_P(type) \
(TYPE_SPECIFIC_FIELD (type) == TYPE_SPECIFIC_GNAT_STUFF \
|| (TYPE_SPECIFIC_FIELD (type) == TYPE_SPECIFIC_NONE \
@@ -1902,21 +1902,21 @@ extern void allocate_gnat_aux_type (struct type *);
(TYPE_SPECIFIC_FIELD (type) = TYPE_SPECIFIC_FIXED_POINT, \
allocate_fixed_point_type_info (type))
-/* * Return the alignment of the type in target addressable memory
+/* Return the alignment of the type in target addressable memory
units, or 0 if no alignment was specified. */
#define TYPE_RAW_ALIGN(thistype) type_raw_align (thistype)
-/* * Return the alignment of the type in target addressable memory
+/* Return the alignment of the type in target addressable memory
units, or 0 if no alignment was specified. */
extern unsigned type_raw_align (struct type *);
-/* * Return the alignment of the type in target addressable memory
+/* Return the alignment of the type in target addressable memory
units. Return 0 if the alignment cannot be determined; but note
that this makes an effort to compute the alignment even it it was
not specified in the debug info. */
extern unsigned type_align (struct type *);
-/* * Set the alignment of the type. The alignment must be a power of
+/* Set the alignment of the type. The alignment must be a power of
2. Returns false if the given value does not fit in the available
space in struct type. */
extern bool set_type_align (struct type *, ULONGEST);
@@ -2096,11 +2096,11 @@ struct builtin_type
/* Pointer types. */
- /* * `pointer to data' type. Some target platforms use an implicitly
+ /* `pointer to data' type. Some target platforms use an implicitly
{sign,zero} -extended 32-bit ABI pointer on a 64-bit ISA. */
struct type *builtin_data_ptr = nullptr;
- /* * `pointer to function (returning void)' type. Harvard
+ /* `pointer to function (returning void)' type. Harvard
architectures mean that ABI function and code pointers are not
interconvertible. Similarly, since ANSI, C standards have
explicitly said that pointers to functions and pointers to data
@@ -2111,28 +2111,28 @@ struct builtin_type
struct type *builtin_func_ptr = nullptr;
- /* * `function returning pointer to function (returning void)' type.
+ /* `function returning pointer to function (returning void)' type.
The final void return type is not significant for it. */
struct type *builtin_func_func = nullptr;
/* Special-purpose types. */
- /* * This type is used to represent a GDB internal function. */
+ /* This type is used to represent a GDB internal function. */
struct type *internal_fn = nullptr;
- /* * This type is used to represent an xmethod. */
+ /* This type is used to represent an xmethod. */
struct type *xmethod = nullptr;
- /* * This type is used to represent symbol addresses. */
+ /* This type is used to represent symbol addresses. */
struct type *builtin_core_addr = nullptr;
- /* * This type represents a type that was unrecognized in symbol
+ /* This type represents a type that was unrecognized in symbol
read-in. */
struct type *builtin_error = nullptr;
- /* * Types used for symbols with no debug information. */
+ /* Types used for symbols with no debug information. */
struct type *nodebug_text_symbol = nullptr;
struct type *nodebug_text_gnu_ifunc_symbol = nullptr;
struct type *nodebug_got_plt_symbol = nullptr;
@@ -2141,11 +2141,11 @@ struct builtin_type
struct type *nodebug_tls_symbol = nullptr;
};
-/* * Return the type table for the specified architecture. */
+/* Return the type table for the specified architecture. */
extern const struct builtin_type *builtin_type (struct gdbarch *gdbarch);
-/* * Return the type table for the specified objfile. */
+/* Return the type table for the specified objfile. */
extern const struct builtin_type *builtin_type (struct objfile *objfile);
@@ -2189,7 +2189,7 @@ extern const struct floatformat *floatformats_bfloat16[BFD_ENDIAN_UNKNOWN];
#define TYPE_ZALLOC(t,size) (memset (TYPE_ALLOC (t, size), 0, size))
-/* * This returns the target type (or NULL) of TYPE, also skipping
+/* This returns the target type (or NULL) of TYPE, also skipping
past typedefs. */
extern struct type *get_target_type (struct type *type);
@@ -2597,7 +2597,7 @@ extern void get_signed_type_minmax (struct type *, LONGEST *, LONGEST *);
extern CORE_ADDR get_pointer_type_max (struct type *);
-/* * Resolve all dynamic values of a type e.g. array bounds to static values.
+/* Resolve all dynamic values of a type e.g. array bounds to static values.
ADDR specifies the location of the variable the type is bound to.
If TYPE has no dynamic properties return TYPE; otherwise a new type with
static properties is returned.
@@ -2615,7 +2615,7 @@ extern struct type *resolve_dynamic_type
(struct type *type, gdb::array_view<const gdb_byte> valaddr,
CORE_ADDR addr, const frame_info_ptr *frame = nullptr);
-/* * Predicate if the type has dynamic values, which are not resolved yet.
+/* Predicate if the type has dynamic values, which are not resolved yet.
See the caveat in 'resolve_dynamic_type' to understand a scenario
where an apparently-resolved type may still be considered
"dynamic". */
@@ -2721,48 +2721,48 @@ extern int is_unique_ancestor (struct type *, struct value *);
/* Overload resolution */
-/* * Badness if parameter list length doesn't match arg list length. */
+/* Badness if parameter list length doesn't match arg list length. */
extern const struct rank LENGTH_MISMATCH_BADNESS;
-/* * Dummy badness value for nonexistent parameter positions. */
+/* Dummy badness value for nonexistent parameter positions. */
extern const struct rank TOO_FEW_PARAMS_BADNESS;
-/* * Badness if no conversion among types. */
+/* Badness if no conversion among types. */
extern const struct rank INCOMPATIBLE_TYPE_BADNESS;
-/* * Badness of an exact match. */
+/* Badness of an exact match. */
extern const struct rank EXACT_MATCH_BADNESS;
-/* * Badness of integral promotion. */
+/* Badness of integral promotion. */
extern const struct rank INTEGER_PROMOTION_BADNESS;
-/* * Badness of floating promotion. */
+/* Badness of floating promotion. */
extern const struct rank FLOAT_PROMOTION_BADNESS;
-/* * Badness of converting a derived class pointer
+/* Badness of converting a derived class pointer
to a base class pointer. */
extern const struct rank BASE_PTR_CONVERSION_BADNESS;
-/* * Badness of integral conversion. */
+/* Badness of integral conversion. */
extern const struct rank INTEGER_CONVERSION_BADNESS;
-/* * Badness of floating conversion. */
+/* Badness of floating conversion. */
extern const struct rank FLOAT_CONVERSION_BADNESS;
-/* * Badness of integer<->floating conversions. */
+/* Badness of integer<->floating conversions. */
extern const struct rank INT_FLOAT_CONVERSION_BADNESS;
-/* * Badness of conversion of pointer to void pointer. */
+/* Badness of conversion of pointer to void pointer. */
extern const struct rank VOID_PTR_CONVERSION_BADNESS;
-/* * Badness of conversion to boolean. */
+/* Badness of conversion to boolean. */
extern const struct rank BOOL_CONVERSION_BADNESS;
-/* * Badness of converting derived to base class. */
+/* Badness of converting derived to base class. */
extern const struct rank BASE_CONVERSION_BADNESS;
-/* * Badness of converting from non-reference to reference. Subrank
+/* Badness of converting from non-reference to reference. Subrank
is the type of reference conversion being done. */
extern const struct rank REFERENCE_CONVERSION_BADNESS;
extern const struct rank REFERENCE_SEE_THROUGH_BADNESS;
-/* * Conversion to rvalue reference. */
+/* Conversion to rvalue reference. */
#define REFERENCE_CONVERSION_RVALUE 1
-/* * Conversion to const lvalue reference. */
+/* Conversion to const lvalue reference. */
#define REFERENCE_CONVERSION_CONST_LVALUE 2
-/* * Badness of converting integer 0 to NULL pointer. */
+/* Badness of converting integer 0 to NULL pointer. */
extern const struct rank NULL_POINTER_CONVERSION;
-/* * Badness of cv-conversion. Subrank is a flag describing the conversions
+/* Badness of cv-conversion. Subrank is a flag describing the conversions
being done. */
extern const struct rank CV_CONVERSION_BADNESS;
#define CV_CONVERSION_CONST 1
@@ -2770,10 +2770,10 @@ extern const struct rank CV_CONVERSION_BADNESS;
/* Non-standard conversions allowed by the debugger */
-/* * Converting a pointer to an int is usually OK. */
+/* Converting a pointer to an int is usually OK. */
extern const struct rank NS_POINTER_CONVERSION_BADNESS;
-/* * Badness of converting a (non-zero) integer constant
+/* Badness of converting a (non-zero) integer constant
to a pointer. */
extern const struct rank NS_INTEGER_POINTER_CONVERSION_BADNESS;
@@ -2839,7 +2839,7 @@ extern bool is_fixed_point_type (struct type *type);
called by INIT_FIXED_POINT_SPECIFIC. */
extern void allocate_fixed_point_type_info (struct type *type);
-/* * When the type includes explicit byte ordering, return that.
+/* When the type includes explicit byte ordering, return that.
Otherwise, the byte ordering from gdbarch_byte_order for
the type's arch is returned. */
diff --git a/gdb/main.h b/gdb/main.h
index a922f0c9c8b..73158cae29d 100644
--- a/gdb/main.h
+++ b/gdb/main.h
@@ -35,7 +35,7 @@ extern int return_child_result_value;
extern int batch_silent;
extern int batch_flag;
-/* * The name of the interpreter if specified on the command line. */
+/* The name of the interpreter if specified on the command line. */
extern std::string interpreter_p;
/* From mingw-hdep.c, used by main.c. */
diff --git a/gdb/type-codes.def b/gdb/type-codes.def
index a8dafba11c4..ebc391c81ae 100644
--- a/gdb/type-codes.def
+++ b/gdb/type-codes.def
@@ -19,7 +19,7 @@
OP (TYPE_CODE_PTR) /**< Pointer type */
-/* * Array type with lower & upper bounds.
+/* Array type with lower & upper bounds.
Regardless of the language, GDB represents multidimensional
array types the way C does: as arrays of arrays. So an
@@ -46,10 +46,10 @@ OP (TYPE_CODE_FLAGS) /**< Bit flags type */
OP (TYPE_CODE_FUNC) /**< Function type */
OP (TYPE_CODE_INT) /**< Integer type */
-/* * Floating type. This is *NOT* a complex type. */
+/* Floating type. This is *NOT* a complex type. */
OP (TYPE_CODE_FLT)
-/* * Void type. The length field specifies the length (probably
+/* Void type. The length field specifies the length (probably
always one) which is used in pointer arithmetic involving
pointers to this type, but actually dereferencing such a
pointer is invalid; a void type has no length and no actual
@@ -60,13 +60,13 @@ OP (TYPE_CODE_VOID)
OP (TYPE_CODE_SET) /**< Pascal sets */
OP (TYPE_CODE_RANGE) /**< Range (integers within spec'd bounds). */
-/* * A string type which is like an array of character but prints
+/* A string type which is like an array of character but prints
differently. It does not contain a length field as Pascal
strings (for many Pascals, anyway) do; if we want to deal with
such strings, we should use a new type code. */
OP (TYPE_CODE_STRING)
-/* * Unknown type. The length field is valid if we were able to
+/* Unknown type. The length field is valid if we were able to
deduce that much about the type, or 0 if we don't even know
that. */
OP (TYPE_CODE_ERROR)
@@ -74,13 +74,13 @@ OP (TYPE_CODE_ERROR)
/* C++ */
OP (TYPE_CODE_METHOD) /**< Method type */
-/* * Pointer-to-member-function type. This describes how to access a
+/* Pointer-to-member-function type. This describes how to access a
particular member function of a class (possibly a virtual
member function). The representation may vary between different
C++ ABIs. */
OP (TYPE_CODE_METHODPTR)
-/* * Pointer-to-member type. This is the offset within a class to
+/* Pointer-to-member type. This is the offset within a class to
some particular data member. The only currently supported
representation uses an unbiased offset, with -1 representing
NULL; this is used by the Itanium C++ ABI (used by GCC on all
@@ -93,7 +93,7 @@ OP (TYPE_CODE_RVALUE_REF) /**< C++ rvalue reference types */
OP (TYPE_CODE_CHAR) /**< *real* character type */
-/* * Boolean type. 0 is false, 1 is true, and other values are
+/* Boolean type. 0 is false, 1 is true, and other values are
non-boolean (e.g. FORTRAN "logical" used as unsigned int). */
OP (TYPE_CODE_BOOL)
@@ -108,16 +108,16 @@ OP (TYPE_CODE_DECFLOAT) /**< Decimal floating point. */
OP (TYPE_CODE_MODULE) /**< Fortran module. */
-/* * Internal function type. */
+/* Internal function type. */
OP (TYPE_CODE_INTERNAL_FUNCTION)
-/* * Methods implemented in extension languages. */
+/* Methods implemented in extension languages. */
OP (TYPE_CODE_XMETHOD)
-/* * Fixed Point type. */
+/* Fixed Point type. */
OP (TYPE_CODE_FIXED_POINT)
-/* * Fortran namelist is a group of variables or arrays that can be
+/* Fortran namelist is a group of variables or arrays that can be
read or written.
Namelist syntax: NAMELIST / groupname / namelist_items ...
diff --git a/gdb/valprint.h b/gdb/valprint.h
index 8e88b964722..15d0be2ac7a 100644
--- a/gdb/valprint.h
+++ b/gdb/valprint.h
@@ -29,7 +29,7 @@ enum val_prettyformat
{
Val_no_prettyformat = 0,
Val_prettyformat,
- /* * Use the default setting which the user has specified. */
+ /* Use the default setting which the user has specified. */
Val_prettyformat_default
};
diff --git a/gdbsupport/common-types.h b/gdbsupport/common-types.h
index fbbe376c769..bd05604dd23 100644
--- a/gdbsupport/common-types.h
+++ b/gdbsupport/common-types.h
@@ -23,10 +23,10 @@
#include <inttypes.h>
#include "gdbsupport/offset-type.h"
-/* * A byte from the program being debugged. */
+/* A byte from the program being debugged. */
typedef unsigned char gdb_byte;
-/* * An address in the program being debugged. Host byte order. */
+/* An address in the program being debugged. Host byte order. */
typedef uint64_t CORE_ADDR;
/* Like a CORE_ADDR, but not directly convertible. This is used to
@@ -38,16 +38,16 @@ DEFINE_OFFSET_TYPE (unrelocated_addr, CORE_ADDR);
typedef int64_t LONGEST;
typedef uint64_t ULONGEST;
-/* * The largest CORE_ADDR value. */
+/* The largest CORE_ADDR value. */
#define CORE_ADDR_MAX (~(CORE_ADDR) 0)
-/* * The largest ULONGEST value, 0xFFFFFFFFFFFFFFFF for 64-bits. */
+/* The largest ULONGEST value, 0xFFFFFFFFFFFFFFFF for 64-bits. */
#define ULONGEST_MAX (~(ULONGEST) 0)
-/* * The largest LONGEST value, 0x7FFFFFFFFFFFFFFF for 64-bits. */
+/* The largest LONGEST value, 0x7FFFFFFFFFFFFFFF for 64-bits. */
#define LONGEST_MAX ((LONGEST) (ULONGEST_MAX >> 1))
-/* * The smallest LONGEST value, 0x8000000000000000 for 64-bits. */
+/* The smallest LONGEST value, 0x8000000000000000 for 64-bits. */
#define LONGEST_MIN ((LONGEST) (~(LONGEST) 0 ^ LONGEST_MAX))
enum tribool { TRIBOOL_UNKNOWN = -1, TRIBOOL_FALSE = 0, TRIBOOL_TRUE = 1 };
diff --git a/gdbsupport/host-defs.h b/gdbsupport/host-defs.h
index fdfbcffc758..bad3931c3a6 100644
--- a/gdbsupport/host-defs.h
+++ b/gdbsupport/host-defs.h
@@ -23,13 +23,13 @@
/* Static host-system-dependent parameters for GDB. */
-/* * Number of bits in a char or unsigned char for the target machine.
+/* Number of bits in a char or unsigned char for the target machine.
Just like CHAR_BIT in <limits.h> but describes the target machine. */
#if !defined (TARGET_CHAR_BIT)
#define TARGET_CHAR_BIT 8
#endif
-/* * If we picked up a copy of CHAR_BIT from a configuration file
+/* If we picked up a copy of CHAR_BIT from a configuration file
(which may get it by including <limits.h>) then use it to set
the number of bits in a host char. If not, use the same size
as the target. */
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 3/4] [gdb] Remove \page marker
2026-06-01 10:53 [PATCH v2 0/4] [gdb] Remove doxygen support Tom de Vries
2026-06-01 10:53 ` [PATCH v2 1/4] [gdb/doc] " Tom de Vries
2026-06-01 10:53 ` [PATCH v2 2/4] [gdb] Remove '/* *' multi-line comment marker Tom de Vries
@ 2026-06-01 10:53 ` Tom de Vries
2026-06-01 10:53 ` [PATCH v2 4/4] [gdb] Remove '/**<' comment marker Tom de Vries
2026-06-01 16:16 ` [PATCH v2 0/4] [gdb] Remove doxygen support Tom Tromey
4 siblings, 0 replies; 6+ messages in thread
From: Tom de Vries @ 2026-06-01 10:53 UTC (permalink / raw)
To: gdb-patches
Remove doxygen \page marker in gdb/gdbtypes.h.
Approved-By: Kevin Buettner <kevinb@redhat.com>
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34186
---
gdb/gdbtypes.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h
index dc3703d74f9..d08cf73711f 100644
--- a/gdb/gdbtypes.h
+++ b/gdb/gdbtypes.h
@@ -22,9 +22,7 @@
#ifndef GDB_GDBTYPES_H
#define GDB_GDBTYPES_H
-/* \page gdbtypes GDB Types
-
- GDB represents all the different kinds of types in programming
+/* GDB represents all the different kinds of types in programming
languages using a common representation defined in gdbtypes.h.
The main data structure is main_type; it consists of a code (such
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 4/4] [gdb] Remove '/**<' comment marker
2026-06-01 10:53 [PATCH v2 0/4] [gdb] Remove doxygen support Tom de Vries
` (2 preceding siblings ...)
2026-06-01 10:53 ` [PATCH v2 3/4] [gdb] Remove \page marker Tom de Vries
@ 2026-06-01 10:53 ` Tom de Vries
2026-06-01 16:16 ` [PATCH v2 0/4] [gdb] Remove doxygen support Tom Tromey
4 siblings, 0 replies; 6+ messages in thread
From: Tom de Vries @ 2026-06-01 10:53 UTC (permalink / raw)
To: gdb-patches
Another comment marker doxygen supports is "/**<". Replace it with "/*".
Suggested-By: Kevin Buettner <kevinb@redhat.com>
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34186
---
gdb/gdbtypes.h | 12 ++++++------
gdb/type-codes.def | 34 +++++++++++++++++-----------------
2 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h
index d08cf73711f..97adbe61db8 100644
--- a/gdb/gdbtypes.h
+++ b/gdb/gdbtypes.h
@@ -63,7 +63,7 @@ struct dwarf2_property_baton;
enum type_code
{
- TYPE_CODE_UNDEF = 0, /**< Not used; catches errors */
+ TYPE_CODE_UNDEF = 0, /* Not used; catches errors */
#define OP(X) X,
#include "type-codes.def"
@@ -483,12 +483,12 @@ struct dynamic_prop_list
enum field_loc_kind
{
- FIELD_LOC_KIND_BITPOS, /**< bitpos */
- FIELD_LOC_KIND_ENUMVAL, /**< enumval */
- FIELD_LOC_KIND_PHYSADDR, /**< physaddr */
- FIELD_LOC_KIND_PHYSNAME, /**< physname */
+ FIELD_LOC_KIND_BITPOS, /* bitpos */
+ FIELD_LOC_KIND_ENUMVAL, /* enumval */
+ FIELD_LOC_KIND_PHYSADDR, /* physaddr */
+ FIELD_LOC_KIND_PHYSNAME, /* physname */
/* A DWARF block that computes the address of the field. */
- FIELD_LOC_KIND_DWARF_BLOCK_ADDR, /**< dwarf_block */
+ FIELD_LOC_KIND_DWARF_BLOCK_ADDR, /* dwarf_block */
/* A DWARF block that computes the bit offset of the field. */
FIELD_LOC_KIND_DWARF_BLOCK_BITPOS,
};
diff --git a/gdb/type-codes.def b/gdb/type-codes.def
index ebc391c81ae..afd2b12d9ba 100644
--- a/gdb/type-codes.def
+++ b/gdb/type-codes.def
@@ -17,7 +17,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/>. */
-OP (TYPE_CODE_PTR) /**< Pointer type */
+OP (TYPE_CODE_PTR) /* Pointer type */
/* Array type with lower & upper bounds.
@@ -39,12 +39,12 @@ OP (TYPE_CODE_PTR) /**< Pointer type */
from right to left, not left to right. */
OP (TYPE_CODE_ARRAY)
-OP (TYPE_CODE_STRUCT) /**< C struct or Pascal record */
-OP (TYPE_CODE_UNION) /**< C union or Pascal variant part */
-OP (TYPE_CODE_ENUM) /**< Enumeration type */
-OP (TYPE_CODE_FLAGS) /**< Bit flags type */
-OP (TYPE_CODE_FUNC) /**< Function type */
-OP (TYPE_CODE_INT) /**< Integer type */
+OP (TYPE_CODE_STRUCT) /* C struct or Pascal record */
+OP (TYPE_CODE_UNION) /* C union or Pascal variant part */
+OP (TYPE_CODE_ENUM) /* Enumeration type */
+OP (TYPE_CODE_FLAGS) /* Bit flags type */
+OP (TYPE_CODE_FUNC) /* Function type */
+OP (TYPE_CODE_INT) /* Integer type */
/* Floating type. This is *NOT* a complex type. */
OP (TYPE_CODE_FLT)
@@ -57,8 +57,8 @@ OP (TYPE_CODE_FLT)
type is a generic pointer. */
OP (TYPE_CODE_VOID)
-OP (TYPE_CODE_SET) /**< Pascal sets */
-OP (TYPE_CODE_RANGE) /**< Range (integers within spec'd bounds). */
+OP (TYPE_CODE_SET) /* Pascal sets */
+OP (TYPE_CODE_RANGE) /* Range (integers within spec'd bounds). */
/* A string type which is like an array of character but prints
differently. It does not contain a length field as Pascal
@@ -72,7 +72,7 @@ OP (TYPE_CODE_STRING)
OP (TYPE_CODE_ERROR)
/* C++ */
-OP (TYPE_CODE_METHOD) /**< Method type */
+OP (TYPE_CODE_METHOD) /* Method type */
/* Pointer-to-member-function type. This describes how to access a
particular member function of a class (possibly a virtual
@@ -87,26 +87,26 @@ OP (TYPE_CODE_METHODPTR)
platforms). */
OP (TYPE_CODE_MEMBERPTR)
-OP (TYPE_CODE_REF) /**< C++ Reference types */
+OP (TYPE_CODE_REF) /* C++ Reference types */
-OP (TYPE_CODE_RVALUE_REF) /**< C++ rvalue reference types */
+OP (TYPE_CODE_RVALUE_REF) /* C++ rvalue reference types */
-OP (TYPE_CODE_CHAR) /**< *real* character type */
+OP (TYPE_CODE_CHAR) /* *real* character type */
/* Boolean type. 0 is false, 1 is true, and other values are
non-boolean (e.g. FORTRAN "logical" used as unsigned int). */
OP (TYPE_CODE_BOOL)
/* Fortran */
-OP (TYPE_CODE_COMPLEX) /**< Complex float */
+OP (TYPE_CODE_COMPLEX) /* Complex float */
OP (TYPE_CODE_TYPEDEF)
-OP (TYPE_CODE_NAMESPACE) /**< C++ namespace. */
+OP (TYPE_CODE_NAMESPACE) /* C++ namespace. */
-OP (TYPE_CODE_DECFLOAT) /**< Decimal floating point. */
+OP (TYPE_CODE_DECFLOAT) /* Decimal floating point. */
-OP (TYPE_CODE_MODULE) /**< Fortran module. */
+OP (TYPE_CODE_MODULE) /* Fortran module. */
/* Internal function type. */
OP (TYPE_CODE_INTERNAL_FUNCTION)
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 0/4] [gdb] Remove doxygen support
2026-06-01 10:53 [PATCH v2 0/4] [gdb] Remove doxygen support Tom de Vries
` (3 preceding siblings ...)
2026-06-01 10:53 ` [PATCH v2 4/4] [gdb] Remove '/**<' comment marker Tom de Vries
@ 2026-06-01 16:16 ` Tom Tromey
4 siblings, 0 replies; 6+ messages in thread
From: Tom Tromey @ 2026-06-01 16:16 UTC (permalink / raw)
To: Tom de Vries; +Cc: gdb-patches
>>>>> "Tom" == Tom de Vries <tdevries@suse.de> writes:
Tom> This patch series removes doxygen support.
Tom> The first patch removes the ability to generate it.
Tom> The second, third and fourth patch remove doxygen-specific annotations.
Tom> Tested on x86_64-linux.
Tom> Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34186
This all looks good to me, thank you for doing this.
Approved-By: Tom Tromey <tom@tromey.com>
Tom
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-06-01 16:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-01 10:53 [PATCH v2 0/4] [gdb] Remove doxygen support Tom de Vries
2026-06-01 10:53 ` [PATCH v2 1/4] [gdb/doc] " Tom de Vries
2026-06-01 10:53 ` [PATCH v2 2/4] [gdb] Remove '/* *' multi-line comment marker Tom de Vries
2026-06-01 10:53 ` [PATCH v2 3/4] [gdb] Remove \page marker Tom de Vries
2026-06-01 10:53 ` [PATCH v2 4/4] [gdb] Remove '/**<' comment marker Tom de Vries
2026-06-01 16:16 ` [PATCH v2 0/4] [gdb] Remove doxygen support Tom Tromey
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox