diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2013-09-05 22:23:42 -0700 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-09-17 02:57:08 +0200 |
commit | a33d9351a88da98e8ea0bf63b9b2c29ecc4a745d (patch) | |
tree | 31390314fae2d8bbfaae11500579b51658a2d75e | |
parent | 90005fae17acc994948aa5a79fc262fd07b69865 (diff) |
Remove useless code setting but not using variables
Clang 3.3 found a variable assigned to itself, probably to cause
GCC to shut up about a variable set-but-unused. So simply stop
assigning the value at all to this variable.
qxcbxsettings.cpp:155:16: error: explicitly assigning a variable of type 'uint' (aka 'unsigned int') to itself [-Werror,-Wself-assign]
Change-Id: I74ae8f724e87c1b3f6b9d358e13d6a440ff4a3e1
Reviewed-by: Uli Schlachter <psychon@znc.in>
Reviewed-by: Jan Arve Sæther <jan-arve.saether@digia.com>
-rw-r--r-- | src/plugins/platforms/xcb/qxcbxsettings.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/plugins/platforms/xcb/qxcbxsettings.cpp b/src/plugins/platforms/xcb/qxcbxsettings.cpp index 1423c6262d..8679d502d9 100644 --- a/src/plugins/platforms/xcb/qxcbxsettings.cpp +++ b/src/plugins/platforms/xcb/qxcbxsettings.cpp @@ -151,8 +151,6 @@ public: // we ignore byteorder for now char byteOrder = xSettings.at(1); Q_UNUSED(byteOrder); - uint serial = *reinterpret_cast<const uint *>(xSettings.mid(4,4).constData()); - serial = serial; uint number_of_settings = *reinterpret_cast<const uint *>(xSettings.mid(8,4).constData()); const char *data = xSettings.constData() + 12; @@ -203,7 +201,6 @@ public: QXcbScreen *screen; xcb_window_t x_settings_window; - int serial; QMap<QByteArray, QXcbXSettingsPropertyValue> settings; }; |