FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Inline expression constructor

Browse source 
The struct expression constructor no longer does any real work, so
this inlines it.  The default destructor can also be used now as well.

gdb/ChangeLog
2021-03-08  Tom Tromey  <tom@tromey.com>

	* parse.c (expression::expression, expression::~expression):
	Remove.
	* expression.h (struct expression): Inline constructor.  Remove
	destructor.
This commit is contained in:
Tom Tromey 2021-03-08 07:27:57 -07:00
parent 79ab486e97
commit b9d06571f9
3 changed files with 13 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
gdb/ChangeLog
View file

@ -1,3 +1,10 @@
2021-03-08 Tom Tromey <tom@tromey.com>
* parse.c (expression::expression, expression::~expression):
Remove.
* expression.h (struct expression): Inline constructor. Remove
destructor.
2021-03-08 Tom Tromey <tom@tromey.com>
* std-operator.def (BINOP_END): Remove.

8
gdb/expression.h
View file

@ -196,8 +196,12 @@ make_operation (Arg... args)
struct expression
{
expression (const struct language_defn *, struct gdbarch *);
~expression ();
expression (const struct language_defn *lang, struct gdbarch *arch)
: language_defn (lang),
gdbarch (arch)
{
}
DISABLE_COPY_AND_ASSIGN (expression);
/* Return the opcode for the outermost sub-expression of this

10
gdb/parse.c
View file

@ -105,16 +105,6 @@ expr_builder::release ()
return std::move (expout);
}
expression::expression (const struct language_defn *lang, struct gdbarch *arch)
: language_defn (lang),
gdbarch (arch)
{
}
expression::~expression ()
{
}
/* Return the type of MSYMBOL, a minimal symbol of OBJFILE. If
ADDRESS_P is not NULL, set it to the MSYMBOL's resolved
address. */