diff options
| author | Tom Tromey <tom@tromey.com> | 2021-08-26 18:17:40 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2021-09-24 11:58:04 -0600 |
| commit | 7ebaa5f7821682c40e79ee1fdfe43528b7d87376 (patch) | |
| tree | e32d94cab5616bedf19aa1637eaea7e1c0df49fb /gdb/valarith.c | |
| parent | 604386598d43e50f183aba65189354e04ffbdda3 (diff) | |
| download | gdb-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/valarith.c')
| -rw-r--r-- | gdb/valarith.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/valarith.c b/gdb/valarith.c index 9ebad64..07472ef 100644 --- a/gdb/valarith.c +++ b/gdb/valarith.c @@ -1653,9 +1653,9 @@ value_binop (struct value *arg1, struct value *arg2, enum exp_opcode op) return val; } -/* Simulate the C operator ! -- return 1 if ARG1 contains zero. */ +/* See value.h. */ -int +bool value_logical_not (struct value *arg1) { int len; |