public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Remove self-assign from make_invisible_and_set_new_height
@ 2019-07-22 14:37 Tom Tromey
0 siblings, 0 replies; only message in thread
From: Tom Tromey @ 2019-07-22 14:37 UTC (permalink / raw)
To: gdb-cvs
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=bfa2a36d94d124eb7b54fd271a543047579b47ee
commit bfa2a36d94d124eb7b54fd271a543047579b47ee
Author: Tom Tromey <tom@tromey.com>
Date: Mon Jul 22 08:34:25 2019 -0600
Remove self-assign from make_invisible_and_set_new_height
In https://sourceware.org/ml/gdb-patches/2019-07/msg00509.html, Jan
pointed out that clang points out that
make_invisible_and_set_new_height self-assigns "height".
This patch fixes the bug by renaming the formal parameter.
gdb/ChangeLog
2019-07-22 Tom Tromey <tom@tromey.com>
* tui/tui-win.c (tui_win_info::make_invisible_and_set_new_height):
Don't self-assign.
Diff:
---
gdb/ChangeLog | 5 +++++
gdb/tui/tui-win.c | 4 ++--
2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 63d6712..bf12f0e 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2019-07-22 Tom Tromey <tom@tromey.com>
+
+ * tui/tui-win.c (tui_win_info::make_invisible_and_set_new_height):
+ Don't self-assign.
+
2019-07-22 Andrew Burgess <andrew.burgess@embecosm.com>
* c-typeprint.c (c_print_typedef): Pass -1 instead of 0 to
diff --git a/gdb/tui/tui-win.c b/gdb/tui/tui-win.c
index db4dc10..785c623 100644
--- a/gdb/tui/tui-win.c
+++ b/gdb/tui/tui-win.c
@@ -1194,10 +1194,10 @@ tui_adjust_win_heights (struct tui_win_info *primary_win_info,
/* See tui-data.h. */
void
-tui_win_info::make_invisible_and_set_new_height (int height)
+tui_win_info::make_invisible_and_set_new_height (int height_)
{
make_visible (false);
- height = height;
+ height = height_;
if (height > 1)
viewport_height = height - 1;
else
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2019-07-22 14:37 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-22 14:37 [binutils-gdb] Remove self-assign from make_invisible_and_set_new_height Tom Tromey
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).