aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@efficios.com>2020-08-31 13:31:01 -0400
committerSimon Marchi <simon.marchi@efficios.com>2020-08-31 13:31:01 -0400
commitf3bd50f1984e83e6abf5e971c56ce8fac3e936db (patch)
treef0a3fc483a60a52f181a05309f740afba5ed2adf
parent22b9b4b05bfad806eb3d16535dcd4dbedb028c40 (diff)
downloadgdb-f3bd50f1984e83e6abf5e971c56ce8fac3e936db.zip
gdb-f3bd50f1984e83e6abf5e971c56ce8fac3e936db.tar.gz
gdb-f3bd50f1984e83e6abf5e971c56ce8fac3e936db.tar.bz2
gdb: fix nits in previous patches
I forgot to fix some nits pointed out in review before merging the "frame inlined in outer frame series", this patch fixes them. gdb/ChangeLog: * frame-unwind.h (frame_prev_register_ftype): Fix adjective ordering in comment. * frame.c (frame_id_eq): Fix indentation. gdb/testsuite/ChangeLog: * gdb.dwarf2/dw2-reg-undefined.exp: Remove spurious #. Change-Id: Iaddde9677fc3f68382558d1a16f5a0b4beb78bac
-rw-r--r--gdb/ChangeLog6
-rw-r--r--gdb/frame-unwind.h2
-rw-r--r--gdb/frame.c2
-rw-r--r--gdb/testsuite/ChangeLog4
-rw-r--r--gdb/testsuite/gdb.dwarf2/dw2-reg-undefined.exp2
5 files changed, 13 insertions, 3 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 9637e4a..cc998d5 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2020-08-31 Simon Marchi <simon.marchi@efficios.com>
+
+ * frame-unwind.h (frame_prev_register_ftype): Fix adjective
+ ordering in comment.
+ * frame.c (frame_id_eq): Fix indentation.
+
2020-08-31 Scott Linder <scott@scottlinder.com>
Simon Marchi <simon.marchi@efficios.com>
diff --git a/gdb/frame-unwind.h b/gdb/frame-unwind.h
index 25a601c..6cc1a88 100644
--- a/gdb/frame-unwind.h
+++ b/gdb/frame-unwind.h
@@ -134,7 +134,7 @@ typedef void (frame_this_id_ftype) (struct frame_info *this_frame,
a register in THIS frame, or a non-lvalue.
If the previous frame's register was not saved by THIS_FRAME and is
- therefore undefined, return a not_lval wholly optimized-out value.
+ therefore undefined, return a wholly optimized-out not_lval value.
THIS_PROLOGUE_CACHE can be used to share any prolog analysis data
with the other unwind methods. Memory for that cache should be
diff --git a/gdb/frame.c b/gdb/frame.c
index 54f4c61..0b708e6 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -774,7 +774,7 @@ frame_id_eq (frame_id l, frame_id r)
bool eq;
if (l.stack_status == FID_STACK_INVALID
- || r.stack_status == FID_STACK_INVALID)
+ || r.stack_status == FID_STACK_INVALID)
/* Like a NaN, if either ID is invalid, the result is false.
Note that a frame ID is invalid iff it is the null frame ID. */
eq = false;
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 66f1fe8..148f31d 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,9 @@
2020-08-31 Simon Marchi <simon.marchi@efficios.com>
+ * gdb.dwarf2/dw2-reg-undefined.exp: Remove spurious #.
+
+2020-08-31 Simon Marchi <simon.marchi@efficios.com>
+
* gdb.dwarf2/frame-inlined-in-outer-frame.exp: New file.
* gdb.dwarf2/frame-inlined-in-outer-frame.S: New file.
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-reg-undefined.exp b/gdb/testsuite/gdb.dwarf2/dw2-reg-undefined.exp
index 4ab7e2db..75ea1f7 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-reg-undefined.exp
+++ b/gdb/testsuite/gdb.dwarf2/dw2-reg-undefined.exp
@@ -76,7 +76,7 @@ for {set f 0} {$f < 3} {incr f} {
}
# Test that the debug log statement in frame_unwind_register_value produces
-# "not saved" and not# "optimized out".
+# "not saved" and not "optimized out".
gdb_test "set debug frame 1"
gdb_test {print $rax} {frame_unwind_register_value[^\r\n]+rax[^\r\n]+not saved.*}
gdb_test "set debug frame 0"