Inline expr_builder methods

This inlines the expr_builder constructor and release method.  These
are straightforward, so this seemed simpler.

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

	* parser-defs.h (struct expr_builder) <expr_builder>: Inline.
	<release>: Inline.
	* parse.c (expr_builder::expr_builder, expr_builder::release):
	Remove.
This commit is contained in:
Tom Tromey 2021-03-08 07:27:57 -07:00
parent b9d06571f9
commit ce284361a2
3 changed files with 15 additions and 16 deletions

View File

@ -1,3 +1,10 @@
2021-03-08 Tom Tromey <tom@tromey.com>
* parser-defs.h (struct expr_builder) <expr_builder>: Inline.
<release>: Inline.
* parse.c (expr_builder::expr_builder, expr_builder::release):
Remove.
2021-03-08 Tom Tromey <tom@tromey.com>
* parse.c (expression::expression, expression::~expression):

View File

@ -91,20 +91,6 @@ innermost_block_tracker::update (const struct block *b,
/* See definition in parser-defs.h. */
expr_builder::expr_builder (const struct language_defn *lang,
struct gdbarch *gdbarch)
: expout (new expression (lang, gdbarch))
{
}
expression_up
expr_builder::release ()
{
return std::move (expout);
}
/* Return the type of MSYMBOL, a minimal symbol of OBJFILE. If
ADDRESS_P is not NULL, set it to the MSYMBOL's resolved
address. */

View File

@ -42,13 +42,19 @@ struct expr_builder
And GDBARCH is the gdbarch to use during parsing. */
expr_builder (const struct language_defn *lang,
struct gdbarch *gdbarch);
struct gdbarch *gdbarch)
: expout (new expression (lang, gdbarch))
{
}
DISABLE_COPY_AND_ASSIGN (expr_builder);
/* Resize the allocated expression to the correct size, and return
it as an expression_up -- passing ownership to the caller. */
ATTRIBUTE_UNUSED_RESULT expression_up release ();
ATTRIBUTE_UNUSED_RESULT expression_up release ()
{
return std::move (expout);
}
/* Return the gdbarch that was passed to the constructor. */