Override

command.cpp

@@ -49,7 +49,7 @@ namespace
 			ParamExtractor(const std::string& prefix
 				,std::vector<std::string>& result):r_prefix(prefix),r_result(result)
 				{}
-			void operator()(const char* value)
+			void operator()(const char* value) override
 				{r_result.push_back(r_prefix + value);}
 		private:
 			const std::string& r_prefix;

dependencybufferdefault.hpp

@@ -14,16 +14,16 @@ namespace Maike
 	class PRIVATE DependencyBufferDefault final:public DependencyBuffer
 		{
 		public:
-			void append(Dependency&& dep)
+			void append(Dependency&& dep) override
 				{m_deps.push_back(std::move(dep));}
 
-			const Dependency* begin() const noexcept
+			const Dependency* begin() const noexcept override
 				{return m_deps.data();}
 
-			const Dependency* end() const noexcept
+			const Dependency* end() const noexcept override
 				{return begin() + m_deps.size();}
 
-			void append(const DependencyBuffer& buffer)
+			void append(const DependencyBuffer& buffer) override
 				{
 				std::for_each(buffer.begin(),buffer.end()
 					,[this](const Dependency& dep)

dependencygraphdefault.hpp

@@ -22,19 +22,19 @@ namespace Maike
 				r_handler(handler),m_patch_needed(0)
 				{}
 
-			DependencyGraphDefault& targetRegister(Handle<Target>&& target);
-			DependencyGraphDefault& targetsPatch();
-			void targetsProcess(TargetProcessorConst&& proc) const;
-			DependencyGraphDefault& targetsProcess(TargetProcessor&& proc);
-			DependencyGraphDefault& targetsRemove(TargetProcessor&& condition);
-			Target* targetFind(const Stringkey& key);
-			const Target* targetFind(const Stringkey& key) const;
+			DependencyGraphDefault& targetRegister(Handle<Target>&& target) override;
+			DependencyGraphDefault& targetsPatch() override;
+			void targetsProcess(TargetProcessorConst&& proc) const override;
+			DependencyGraphDefault& targetsProcess(TargetProcessor&& proc) override;
+			DependencyGraphDefault& targetsRemove(TargetProcessor&& condition) override;
+			Target* targetFind(const Stringkey& key) override;
+			const Target* targetFind(const Stringkey& key) const override;
 
-			const Twins<size_t>& idRangeGet() const;
+			const Twins<size_t>& idRangeGet() const override;
 
-			DependencyGraphDefault& clear() noexcept;
+			DependencyGraphDefault& clear() noexcept override;
 
-			size_t targetsCountGet() const noexcept
+			size_t targetsCountGet() const noexcept override
 				{return m_targets.size();}
 
 		private:

filein.hpp

@@ -18,15 +18,15 @@ namespace Maike
 			explicit FileIn(const char* filename);
 			~FileIn() noexcept;
 
-			size_t read(void* buffer,size_t count);
+			size_t read(void* buffer,size_t count) override;
 
-			const char* nameGet() const noexcept
+			const char* nameGet() const noexcept override
 				{return m_name.c_str();}
 
 		private:
 			std::string m_name;
 			intptr_t m_handle;
-			void destroy()
+			void destroy() override
 				{delete this;}
 		};
 	}

fileout.hpp

@@ -25,11 +25,11 @@ namespace Maike
 			FileOut(StdStream stream);
 			~FileOut() noexcept;
 
-			size_t write(const void* buffer,size_t count);
+			size_t write(const void* buffer,size_t count) override;
 
 		private:
 			intptr_t m_handle;
-			void destroy()
+			void destroy() override
 				{delete this;}
 		};
 	}

graphedgewriterdot.hpp

@@ -16,7 +16,7 @@ namespace Maike
 		{
 		public:
 			GraphEdgeWriterDOT(WriteBuffer& wb);
-			void edgeWrite(const char* from,const char* to,const char* label);
+			void edgeWrite(const char* from,const char* to,const char* label) override;
 			~GraphEdgeWriterDOT();
 		private:
 			WriteBuffer& r_wb;

maike-main.cpp

@@ -194,18 +194,18 @@ namespace
 			StringReader(const char* str,size_t n):r_str(str),m_n(n)
 				{}
 
-			size_t read(void* buffer,size_t n)
+			size_t read(void* buffer,size_t n) override
 				{
 				auto n_read=std::min(n,m_n);
 				memcpy(buffer,r_str,n_read);
 				m_n-=n_read;
 				return n_read;
 				}
 
-			const char* nameGet() const noexcept
+			const char* nameGet() const noexcept override
 				{return "Command line buffer";}
 
-			void destroy(){}
+			void destroy() override {}
 
 		private:
 			const char* r_str;

maike.cpp

@@ -215,7 +215,7 @@ namespace
 			TargetsListAll(WriteBuffer& wb):r_wb(wb)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				r_wb.write(" * ").write(target.nameGet());
 				auto description=target.descriptionGet();
@@ -247,7 +247,7 @@ namespace
 			TargetsListLeaf(WriteBuffer& wb):r_wb(wb)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				auto x=target.dependenciesInverseGet();
 				if(x.second - x.first==0)
@@ -257,7 +257,7 @@ namespace
 					if(*description!='\0')
 						{r_wb.write(" -- ").write(description);}
 					r_wb.write("\n");
-					}	
+					}
 				return 0;
 				}
 		private:
@@ -283,7 +283,7 @@ namespace
 			TargetsListExternal(WriteBuffer& wb):r_wb(wb)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				auto ptr=dynamic_cast<const TargetPlaceholder*>(&target);
 				if(ptr!=nullptr && ptr->relation()!=Dependency::Relation::MISC)
@@ -399,7 +399,7 @@ namespace
 				,id_range(maike.targetIdRangeGet())
 				{}
 
-			int operator()(DependencyGraph&,Target& target)
+			int operator()(DependencyGraph&,Target& target) override
 				{
 				auto x=target.dependenciesInverseGet();
 				if(x.second - x.first==0)
@@ -436,7 +436,7 @@ namespace
 			GraphEdgesWrite(GraphEdgeWriter& writer):r_writer(writer)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				auto name_target=target.nameGet();
 				if(strcmp(name_target,".")!=0)
@@ -539,7 +539,7 @@ namespace
 			TargetDumpJSON(ResourceObject& db):r_db(db)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				auto target_obj=r_db.createObject();
 				target.dump(target_obj);
@@ -594,7 +594,7 @@ namespace
 			TargetDumpTSV(WriteBuffer& wb):r_wb(wb)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				::targetDumpTSV(target,r_wb);
 				return 0;
@@ -648,7 +648,7 @@ namespace
 				,id_range(maike.targetIdRangeGet())
 				{}
 
-			int operator()(DependencyGraph&,Target& target)
+			int operator()(DependencyGraph&,Target& target) override
 				{
 				auto x=target.dependenciesInverseGet();
 				if(x.second - x.first==0)
@@ -682,7 +682,7 @@ namespace
 				r_target_dir(target_dir),r_target_list(target_list)
 				{}
 
-			int operator()(const DependencyGraph&,const Target& target)
+			int operator()(const DependencyGraph&,const Target& target) override
 				{
 				r_target_list.insert(Stringkey(dircat(r_target_dir,target.nameGet()).c_str()));
 				return 0;

maikeconfig.json

@@ -23,6 +23,7 @@
                         "-Wall",
                         "-Wsuggest-final-types",
 						"-Wextra",
+						"-Wsuggest-override",
 						"-Werror",
                         "{cflags_extra}",
                         "{iquote}",
@@ -50,6 +51,7 @@
                         "-Wall",
                         "-Wsuggest-final-types",
 						"-Wextra",
+						"-Wsuggest-override",
 						"-Werror",
                         "{cflags_extra}",
                         "{iquote}",
@@ -96,6 +98,7 @@
                         "-Wsuggest-final-types",
                         "-Wunused-parameter",
 						"-Wextra",
+						"-Wsuggest-override",
 						"-Werror",
                         "{cflags_extra}",
                         "{iquote}",

parametersetmapfixed.hpp

@@ -27,7 +27,7 @@ namespace Maike
 			using BaseType::length;
 			using BaseType::end;
 
-			void parameterGet(const Stringkey& key,ParameterProcessor&& proc) const;
+			void parameterGet(const Stringkey& key,ParameterProcessor&& proc) const override;
 		};
 
 	template<Stringkey::HashValue ... args>

pipe.hpp

@@ -89,8 +89,8 @@ namespace Maike
 					void operator delete(void *)=delete;
 					void operator delete[](void*)=delete;
 
-					size_t read(void* buffer, size_t n);
-					const char* nameGet() const noexcept;
+					size_t read(void* buffer, size_t n) override;
+					const char* nameGet() const noexcept override;
 
 					void init(intptr_t handle) noexcept
 						{m_handle=handle;}
@@ -99,7 +99,7 @@ namespace Maike
 
 				private:
 					intptr_t m_handle;
-					void destroy() noexcept
+					void destroy() noexcept override
 						{close();}
 				};
 
@@ -119,7 +119,7 @@ namespace Maike
 					void operator delete(void *)=delete;
 					void operator delete[](void*)=delete;
 
-					size_t write(const void* buffer, size_t n);
+					size_t write(const void* buffer, size_t n) override;
 
 					void init(intptr_t handle,bool echo) noexcept
 						{m_handle=handle; m_echo=echo;}
@@ -129,7 +129,7 @@ namespace Maike
 				private:
 					intptr_t m_handle;
 					bool m_echo;
-					void destroy() noexcept
+					void destroy() noexcept override
 						{close();}
 				};
 

session.cpp

@@ -179,7 +179,7 @@ namespace
 		public:
 			BySourceName(const char* filename):r_filename(filename){}
 
-			int operator()(DependencyGraph&,Target& target)
+			int operator()(DependencyGraph&,Target& target) override
 				{return strcmp(target.sourceNameGet(),r_filename);}
 
 		private:
@@ -191,7 +191,7 @@ namespace
 		public:
 			ByParentDirectory(const char* filename):r_filename(filename){}
 
-			int operator()(DependencyGraph&,Target& target)
+			int operator()(DependencyGraph&,Target& target) override
 				{return strcmp(target.inDirGet(),r_filename);}
 
 		private:
@@ -260,7 +260,7 @@ namespace
 				r_delegator(delegator)
 				{}
 
-			void operator()(const Stringkey& filename_ext,const Target_Hook& hook)
+			void operator()(const Stringkey& filename_ext,const Target_Hook& hook) override
 				{
 				r_delegator.loaderRegister(filename_ext,hook.loaderGet());
 				}

spiderdefault.hpp

@@ -24,8 +24,8 @@ namespace Maike
 			SpiderDefault(Target_FactoryDelegator& deleg
 				,DependencyGraph& targets);
 
-			SpiderDefault& scanFile(const char* filename,const char* in_dir);
-			SpiderDefault& run();
+			SpiderDefault& scanFile(const char* filename,const char* in_dir) override;
+			SpiderDefault& run() override;
 
 		private:
 			Target_FactoryDelegator& r_target_creator;

systemtargetinfo.hpp

@@ -24,7 +24,7 @@ namespace Maike
 			~SystemTargetInfo() noexcept;
 
 			void parameterGet(const Stringkey& key
-				,ParameterProcessor&& proc) const;
+				,ParameterProcessor&& proc) const override;
 
 			Variant variableGet(const Stringkey& key) const noexcept;
 
@@ -47,8 +47,8 @@ namespace Maike
 			void clear();
 
 			SystemTargetInfo& configAppend(const ResourceObject& targetinfo);
-			void configDump(ResourceObject& targetinfo) const;
-			ResourceObject configDump() const;
+			void configDump(ResourceObject& targetinfo) const override;
+			ResourceObject configDump() const override;
 
 			SystemTargetInfo& sysvarsLoad();
 			SystemTargetInfo& configAppendDefault();