* [PATCH] Fix some value comments
@ 2023-03-01 21:30 Tom Tromey
2023-03-02 9:29 ` Andrew Burgess
0 siblings, 1 reply; 2+ messages in thread
From: Tom Tromey @ 2023-03-01 21:30 UTC (permalink / raw)
To: gdb-patches; +Cc: Tom Tromey
I noticed a very stale comment in valarith.c. This patch fixes a few
comments in this area.
---
gdb/valarith.c | 6 +++++-
gdb/value.h | 5 +++++
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/gdb/valarith.c b/gdb/valarith.c
index 6ea0cd9e8d1..4c2b44360bb 100644
--- a/gdb/valarith.c
+++ b/gdb/valarith.c
@@ -1884,7 +1884,7 @@ value_less (struct value *arg1, struct value *arg2)
}
}
\f
-/* The unary operators +, - and ~. They free the argument ARG1. */
+/* See value.h. */
struct value *
value_pos (struct value *arg1)
@@ -1902,6 +1902,8 @@ value_pos (struct value *arg1)
error (_("Argument to positive operation not a number."));
}
+/* See value.h. */
+
struct value *
value_neg (struct value *arg1)
{
@@ -1948,6 +1950,8 @@ value_neg (struct value *arg1)
error (_("Argument to negate operation not a number."));
}
+/* See value.h. */
+
struct value *
value_complement (struct value *arg1)
{
diff --git a/gdb/value.h b/gdb/value.h
index c3eb96bf143..d83c4ab3674 100644
--- a/gdb/value.h
+++ b/gdb/value.h
@@ -1206,10 +1206,15 @@ extern struct value *value_ref (struct value *arg1, enum type_code refcode);
extern struct value *value_assign (struct value *toval,
struct value *fromval);
+/* The unary + operation. */
extern struct value *value_pos (struct value *arg1);
+/* The unary - operation. */
extern struct value *value_neg (struct value *arg1);
+/* The unary ~ operation -- but note that it also implements the GCC
+ extension, where ~ of a complex number is the complex
+ conjugate. */
extern struct value *value_complement (struct value *arg1);
extern struct value *value_struct_elt (struct value **argp,
--
2.39.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix some value comments
2023-03-01 21:30 [PATCH] Fix some value comments Tom Tromey
@ 2023-03-02 9:29 ` Andrew Burgess
0 siblings, 0 replies; 2+ messages in thread
From: Andrew Burgess @ 2023-03-02 9:29 UTC (permalink / raw)
To: Tom Tromey via Gdb-patches, gdb-patches; +Cc: Tom Tromey
Tom Tromey via Gdb-patches <gdb-patches@sourceware.org> writes:
> I noticed a very stale comment in valarith.c. This patch fixes a few
> comments in this area.
LGTM.
Reviewed-By: Andrew Burgess <aburgess@redhat.com>
Thanks,
Andrew
> ---
> gdb/valarith.c | 6 +++++-
> gdb/value.h | 5 +++++
> 2 files changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/gdb/valarith.c b/gdb/valarith.c
> index 6ea0cd9e8d1..4c2b44360bb 100644
> --- a/gdb/valarith.c
> +++ b/gdb/valarith.c
> @@ -1884,7 +1884,7 @@ value_less (struct value *arg1, struct value *arg2)
> }
> }
> \f
> -/* The unary operators +, - and ~. They free the argument ARG1. */
> +/* See value.h. */
>
> struct value *
> value_pos (struct value *arg1)
> @@ -1902,6 +1902,8 @@ value_pos (struct value *arg1)
> error (_("Argument to positive operation not a number."));
> }
>
> +/* See value.h. */
> +
> struct value *
> value_neg (struct value *arg1)
> {
> @@ -1948,6 +1950,8 @@ value_neg (struct value *arg1)
> error (_("Argument to negate operation not a number."));
> }
>
> +/* See value.h. */
> +
> struct value *
> value_complement (struct value *arg1)
> {
> diff --git a/gdb/value.h b/gdb/value.h
> index c3eb96bf143..d83c4ab3674 100644
> --- a/gdb/value.h
> +++ b/gdb/value.h
> @@ -1206,10 +1206,15 @@ extern struct value *value_ref (struct value *arg1, enum type_code refcode);
> extern struct value *value_assign (struct value *toval,
> struct value *fromval);
>
> +/* The unary + operation. */
> extern struct value *value_pos (struct value *arg1);
>
> +/* The unary - operation. */
> extern struct value *value_neg (struct value *arg1);
>
> +/* The unary ~ operation -- but note that it also implements the GCC
> + extension, where ~ of a complex number is the complex
> + conjugate. */
> extern struct value *value_complement (struct value *arg1);
>
> extern struct value *value_struct_elt (struct value **argp,
> --
> 2.39.1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-03-02 9:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-01 21:30 [PATCH] Fix some value comments Tom Tromey
2023-03-02 9:29 ` Andrew Burgess
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).