From ddd44f39501b32ca1b02dd5f83d3cd2e582ec345 Mon Sep 17 00:00:00 2001 From: Harry van Haaren Date: Tue, 10 Dec 2013 12:40:20 +0000 Subject: [PATCH] -Added -Werror to build flags, fixed two offenders --- src/CMakeLists.txt | 4 ++-- src/gui.cxx | 2 +- src/observer/midi.cxx | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 6b552a2..706c812 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -37,8 +37,8 @@ IF(BUILD_TESTS) ENDIF(BUILD_TESTS) IF(RELEASE_BUILD) - SET(CMAKE_CXX_FLAGS "-g -Wall -Wno-unused-variable -ffast-math") - SET(CMAKE_C_FLAGS "-g -Wall -W -Wno-unused-variable -ffast-math") + SET(CMAKE_CXX_FLAGS "-g -Wall -Werror -Wno-unused-variable -ffast-math") + SET(CMAKE_C_FLAGS "-g -Wall -Werror -W -Wno-unused-variable -ffast-math") ENDIF(RELEASE_BUILD) diff --git a/src/gui.cxx b/src/gui.cxx index 820c8ad..1e1471f 100644 --- a/src/gui.cxx +++ b/src/gui.cxx @@ -475,7 +475,7 @@ void Gui::addMidiControllerToSetup(std::string c) void Gui::setupMidiControllers() { - for(int i = 0; i < controllerVector.size(); i++) + for(unsigned int i = 0; i < controllerVector.size(); i++) { GenericMIDI* c = new GenericMIDI( controllerVector.at(i).c_str() ); if ( c ) diff --git a/src/observer/midi.cxx b/src/observer/midi.cxx index 51d5766..e730157 100644 --- a/src/observer/midi.cxx +++ b/src/observer/midi.cxx @@ -25,9 +25,9 @@ extern Jack* jack; MidiIO::MidiIO() : + portsRegistered(false), jackInputPort(0), - jackOutputPort(0), - portsRegistered(false) + jackOutputPort(0) { //LUPPP_NOTE("MidiIO %i",this); }