aboutsummaryrefslogtreecommitdiff
path: root/gdb/eval.c
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2021-08-26 18:17:40 -0600
committerTom Tromey <tom@tromey.com>2021-09-24 11:58:04 -0600
commit7ebaa5f7821682c40e79ee1fdfe43528b7d87376 (patch)
treee32d94cab5616bedf19aa1637eaea7e1c0df49fb /gdb/eval.c
parent604386598d43e50f183aba65189354e04ffbdda3 (diff)
downloadgdb-7ebaa5f7821682c40e79ee1fdfe43528b7d87376.zip
gdb-7ebaa5f7821682c40e79ee1fdfe43528b7d87376.tar.gz
gdb-7ebaa5f7821682c40e79ee1fdfe43528b7d87376.tar.bz2
Move value_true to value.h
I noticed that value_true is declared in language.h and defined in language.c. However, as part of the value API, I think it would be better in one of those files. And, because it is very short, I changed it to be an inline function in value.h. I've also removed a comment from the implementation, on the basis that it seems obsolete -- if the change it suggests was needed, it probably would have been done by now; and if it is needed in the future, odds are it would be done differently anyway. Finally, this patch also changes value_true and value_logical_not to return a bool, and updates some uses.
Diffstat (limited to 'gdb/eval.c')
-rw-r--r--gdb/eval.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/eval.c b/gdb/eval.c
index 1c5c8cf..bfab6d8 100644
--- a/gdb/eval.c
+++ b/gdb/eval.c
@@ -2204,7 +2204,7 @@ logical_and_operation::evaluate (struct type *expect_type,
}
else
{
- int tem = value_logical_not (arg1);
+ bool tem = value_logical_not (arg1);
if (!tem)
{
arg2 = std::get<1> (m_storage)->evaluate (nullptr, exp, noside);
@@ -2233,7 +2233,7 @@ logical_or_operation::evaluate (struct type *expect_type,
}
else
{
- int tem = value_logical_not (arg1);
+ bool tem = value_logical_not (arg1);
if (tem)
{
arg2 = std::get<1> (m_storage)->evaluate (nullptr, exp, noside);