From e921a2492929c9ab623572d0f0bda1997c0a4d94 Mon Sep 17 00:00:00 2001 From: Stefano Sanfilippo Date: Sat, 7 Mar 2015 17:24:25 +0100 Subject: [PATCH] Adding -Wextra -Werror to compiler flags and fixing issues. --- BitcodeEmitter.cpp | 4 +++- CMakeLists.txt | 4 +++- main.cpp | 2 +- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/BitcodeEmitter.cpp b/BitcodeEmitter.cpp index 57c1c71..80f798c 100644 --- a/BitcodeEmitter.cpp +++ b/BitcodeEmitter.cpp @@ -129,6 +129,8 @@ llvm::Type *LLVMType(Type const& type) { case Type::UNKNOWN: return nullptr; // FIXME } + + return nullptr; } static @@ -355,7 +357,7 @@ bool BitcodeEmitter::emit(Input const& node) { return true; } -bool BitcodeEmitter::emit(Abort const& node) { +bool BitcodeEmitter::emit(Abort const&) { llvm::Function *callee = module->getFunction(ABORT_NAME); if (callee == nullptr) { diff --git a/CMakeLists.txt b/CMakeLists.txt index 96dc15b..481ea2c 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -85,7 +85,9 @@ include_directories( ${CMAKE_CURRENT_SOURCE_DIR} ) -add_definitions(-Wall -Wno-deprecated-register -std=c++0x -DYYDEBUG=0 -O2) +add_definitions( + -Wall -Wextra -Werror -Wno-deprecated-register -std=c++0x -DYYDEBUG=0 -O2 +) bison_target(Parser Monicelli.ypp ${CMAKE_CURRENT_BINARY_DIR}/Parser.cpp) flex_target(Scanner Monicelli.lpp ${CMAKE_CURRENT_BINARY_DIR}/Lexer.cpp) diff --git a/main.cpp b/main.cpp index 70ef9e8..c30546c 100644 --- a/main.cpp +++ b/main.cpp @@ -26,7 +26,7 @@ using namespace monicelli; -int main(int argc, char **argv) { +int main() { Program program; Scanner scanner(&std::cin); Parser parser(scanner, program);