Add -Wshadow and solve warnings
This commit is contained in:
@@ -7,3 +7,5 @@ include(common-msvc.pri)
|
|||||||
|
|
||||||
#clang-msvc does not enable the c++17 flag with qmake's c++1z flag.
|
#clang-msvc does not enable the c++17 flag with qmake's c++1z flag.
|
||||||
QMAKE_CXXFLAGS += /std:c++17
|
QMAKE_CXXFLAGS += /std:c++17
|
||||||
|
|
||||||
|
QMAKE_CXXFLAGS += -Xclang -Wshadow
|
||||||
|
|||||||
@@ -1,3 +1,5 @@
|
|||||||
QMAKE_CXXFLAGS += -Werror
|
QMAKE_CXXFLAGS += -Werror
|
||||||
#All includes from the third-party directory will not warn.
|
#All includes from the third-party directory will not warn.
|
||||||
QMAKE_CXXFLAGS += --system-header-prefix=third-party
|
QMAKE_CXXFLAGS += --system-header-prefix=third-party
|
||||||
|
|
||||||
|
QMAKE_CXXFLAGS += -Wshadow
|
||||||
|
|||||||
@@ -1,3 +1,6 @@
|
|||||||
#g++-7 is needed for C++17 features. travis does not supply this by default.
|
#g++-7 is needed for C++17 features. travis does not supply this by default.
|
||||||
QMAKE_CXX = g++-7
|
QMAKE_CXX = g++-7
|
||||||
|
|
||||||
QMAKE_CXXFLAGS += -Werror
|
QMAKE_CXXFLAGS += -Werror
|
||||||
|
|
||||||
|
QMAKE_CXXFLAGS += -Wshadow
|
||||||
|
|||||||
@@ -64,8 +64,8 @@ void UtilitiesTabController::initStage1()
|
|||||||
void UtilitiesTabController::initStage2( OverlayController* var_parent,
|
void UtilitiesTabController::initStage2( OverlayController* var_parent,
|
||||||
QQuickWindow* var_widget )
|
QQuickWindow* var_widget )
|
||||||
{
|
{
|
||||||
this->parent = var_parent;
|
this->m_parent = var_parent;
|
||||||
this->widget = var_widget;
|
this->m_widget = var_widget;
|
||||||
}
|
}
|
||||||
|
|
||||||
void UtilitiesTabController::sendKeyboardInput( QString input )
|
void UtilitiesTabController::sendKeyboardInput( QString input )
|
||||||
@@ -467,7 +467,7 @@ void UtilitiesTabController::eventLoopTick()
|
|||||||
}
|
}
|
||||||
*/
|
*/
|
||||||
auto nError = vrnotification->CreateNotification(
|
auto nError = vrnotification->CreateNotification(
|
||||||
parent->overlayHandle(),
|
m_parent->overlayHandle(),
|
||||||
666,
|
666,
|
||||||
vr::EVRNotificationType_Transient,
|
vr::EVRNotificationType_Transient,
|
||||||
alarmMessageBuffer,
|
alarmMessageBuffer,
|
||||||
|
|||||||
@@ -33,8 +33,8 @@ class UtilitiesTabController : public QObject
|
|||||||
setSteamDesktopOverlayWidth NOTIFY steamDesktopOverlayWidthChanged )
|
setSteamDesktopOverlayWidth NOTIFY steamDesktopOverlayWidthChanged )
|
||||||
|
|
||||||
private:
|
private:
|
||||||
OverlayController* parent;
|
OverlayController* m_parent;
|
||||||
QQuickWindow* widget;
|
QQuickWindow* m_widget;
|
||||||
|
|
||||||
unsigned settingsUpdateCounter = 0;
|
unsigned settingsUpdateCounter = 0;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user