diff options
| author | Tom Tromey <tromey@adacore.com> | 2022-04-07 10:17:02 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2022-04-14 10:04:13 -0600 |
| commit | 4536b3bb61fd2fc7b3482cdc8d03a2542cdf5d9e (patch) | |
| tree | 83928b73d3bd6d9c8164a719eb85951f76f9c9dd /gdb/target | |
| parent | 7ae6857316e020467f84645ba7c1c5ec6c085bd9 (diff) | |
| download | binutils-4536b3bb61fd2fc7b3482cdc8d03a2542cdf5d9e.zip binutils-4536b3bb61fd2fc7b3482cdc8d03a2542cdf5d9e.tar.gz binutils-4536b3bb61fd2fc7b3482cdc8d03a2542cdf5d9e.tar.bz2 | |
Silence -Wmaybe-uninitialized warning from target_waitstatus
Currently, one use of target_waitstatus yields a warning:
target/waitstatus.h: In function 'void stop_all_threads()':
target/waitstatus.h:175:13: warning: 'ws.target_waitstatus::m_value' may be used uninitialized in this function [-Wmaybe-uninitialized]
175 | m_value = other.m_value;
| ~~~~~~~~^~~~~~~~~~~~~~~
This patch silences the warning. I tried the "volatile member"
approach that was used for gdb::optional, but that didn't work, so
this patch simply initializes the member.
Diffstat (limited to 'gdb/target')
| -rw-r--r-- | gdb/target/waitstatus.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/target/waitstatus.h b/gdb/target/waitstatus.h index 7b5faed8..63bbd73 100644 --- a/gdb/target/waitstatus.h +++ b/gdb/target/waitstatus.h @@ -418,7 +418,7 @@ private: char *execd_pathname; /* Syscall number */ int syscall_number; - } m_value; + } m_value {}; }; /* Extended reasons that can explain why a target/thread stopped for a |