Compile flags and option for debug. Removed dead line of code.

Author: LinuxDevon

CMakeLists.txt

@@ -2,12 +2,23 @@ cmake_minimum_required(VERSION 3.10)
 
 project(dbc)
 
+option(DEBUG "use debug flag" NO)
+
 # specify the C++ standard
 set(CMAKE_CXX_STANDARD 11)
 set(CMAKE_CXX_STANDARD_REQUIRED True)
 
-# Add in the debug flag0.
-set(GCC_COVERAGE_COMPILE_FLAGS "-g")
+set(GCC_COMPILE_FLAGS "-Wextra -Wall -Wfloat-equal -Wundef -Wshadow \
+-Wpointer-arith -Wcast-align -Wstrict-prototypes -Wwrite-strings \
+-Waggregate-return -Wcast-qual -Wswitch-default -Wswitch-enum -Wconversion \
+-Wunreachable-code -Wformat=2 -Werror -Wuninitialized -Winit-self")
+
+if(DEBUG)
+	set(GCC_COMPILE_FLAGS ${GCC_COMPILE_FLAGS}" -g")
+else()
+	set(GCC_COMPILE_FLAGS ${GCC_COMPILE_FLAGS}" -O2")
+endif()
+
 set(CMAKE_CXX_FLAGS  "${CMAKE_CXX_FLAGS} ${GCC_COVERAGE_COMPILE_FLAGS}")
 
 # add where to find the source files

src/dbc.hpp

@@ -2,7 +2,6 @@
 #include "util/utils.hpp"
 
 #include <regex>
-#include <iostream>
 
 namespace libdbc {
 
@@ -51,8 +50,6 @@ namespace libdbc {
 		const std::regex name_space_re;
 
 		void parse_dbc_header(std::istream& file_stream) {
-			std::vector<std::string> lines;
-
 			std::string line;
 			std::smatch match;
 			bool is_blank = true;