diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2022-01-31 16:27:39 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2022-01-31 16:27:42 +0100 |
| commit | 0ff13913287b3874e305535c78bc43b190217168 (patch) | |
| tree | 873803adb870742d419f45130b8f6f31c404d4d5 /src | |
| parent | ad05e68e1726baf2b2bb4a7b9a034458bb522f8a (diff) | |
| parent | 1111d33532516c16fb2e22660ac2745ce56ad6cd (diff) | |
| download | bitcoin-0ff13913287b3874e305535c78bc43b190217168.tar.xz | |
Merge bitcoin/bitcoin#24191: refactor: Make MessageBoxFlags enum underlying type unsigned
1111d33532516c16fb2e22660ac2745ce56ad6cd refactor: Make MessageBoxFlags enum underlying type unsigned (MarcoFalke)
Pull request description:
All values in the enum are unsigned. Also, flags shouldn't be treated as signed types. So clarify the underlying type and remove a sanitizer suppression.
ACKs for top commit:
hebasto:
ACK 1111d33532516c16fb2e22660ac2745ce56ad6cd, I have reviewed the code and it looks OK, I agree it can be merged.
Tree-SHA512: 48b16c4a0ace1a1e4d351d6eadadbb1bc42aef7fd82e24e3ea50c62f2c04a552ed21027158d09aa97e630c8c7d732cb150c38065333d7c2accbae46593b7ed9f
Diffstat (limited to 'src')
| -rw-r--r-- | src/node/ui_interface.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/node/ui_interface.h b/src/node/ui_interface.h index 5c7c3e7074..d02238b549 100644 --- a/src/node/ui_interface.h +++ b/src/node/ui_interface.h @@ -25,8 +25,7 @@ class CClientUIInterface { public: /** Flags for CClientUIInterface::ThreadSafeMessageBox */ - enum MessageBoxFlags - { + enum MessageBoxFlags : uint32_t { ICON_INFORMATION = 0, ICON_WARNING = (1U << 0), ICON_ERROR = (1U << 1), |