forked from eden-emu/eden
		
	Merge pull request #6744 from lioncash/exc
exception: Make constructors explicit
This commit is contained in:
		
						commit
						b944f2b0d3
					
				
					 1 changed files with 6 additions and 6 deletions
				
			
		|  | @ -4,7 +4,7 @@ | ||||||
| 
 | 
 | ||||||
| #pragma once | #pragma once | ||||||
| 
 | 
 | ||||||
| #include <stdexcept> | #include <exception> | ||||||
| #include <string> | #include <string> | ||||||
| #include <string_view> | #include <string_view> | ||||||
| #include <utility> | #include <utility> | ||||||
|  | @ -17,7 +17,7 @@ class Exception : public std::exception { | ||||||
| public: | public: | ||||||
|     explicit Exception(std::string message) noexcept : err_message{std::move(message)} {} |     explicit Exception(std::string message) noexcept : err_message{std::move(message)} {} | ||||||
| 
 | 
 | ||||||
|     const char* what() const noexcept override { |     [[nodiscard]] const char* what() const noexcept override { | ||||||
|         return err_message.c_str(); |         return err_message.c_str(); | ||||||
|     } |     } | ||||||
| 
 | 
 | ||||||
|  | @ -36,21 +36,21 @@ private: | ||||||
| class LogicError : public Exception { | class LogicError : public Exception { | ||||||
| public: | public: | ||||||
|     template <typename... Args> |     template <typename... Args> | ||||||
|     LogicError(const char* message, Args&&... args) |     explicit LogicError(const char* message, Args&&... args) | ||||||
|         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} {} |         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} {} | ||||||
| }; | }; | ||||||
| 
 | 
 | ||||||
| class RuntimeError : public Exception { | class RuntimeError : public Exception { | ||||||
| public: | public: | ||||||
|     template <typename... Args> |     template <typename... Args> | ||||||
|     RuntimeError(const char* message, Args&&... args) |     explicit RuntimeError(const char* message, Args&&... args) | ||||||
|         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} {} |         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} {} | ||||||
| }; | }; | ||||||
| 
 | 
 | ||||||
| class NotImplementedException : public Exception { | class NotImplementedException : public Exception { | ||||||
| public: | public: | ||||||
|     template <typename... Args> |     template <typename... Args> | ||||||
|     NotImplementedException(const char* message, Args&&... args) |     explicit NotImplementedException(const char* message, Args&&... args) | ||||||
|         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} { |         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} { | ||||||
|         Append(" is not implemented"); |         Append(" is not implemented"); | ||||||
|     } |     } | ||||||
|  | @ -59,7 +59,7 @@ public: | ||||||
| class InvalidArgument : public Exception { | class InvalidArgument : public Exception { | ||||||
| public: | public: | ||||||
|     template <typename... Args> |     template <typename... Args> | ||||||
|     InvalidArgument(const char* message, Args&&... args) |     explicit InvalidArgument(const char* message, Args&&... args) | ||||||
|         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} {} |         : Exception{fmt::format(fmt::runtime(message), std::forward<Args>(args)...)} {} | ||||||
| }; | }; | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Rodrigo Locatti
						Rodrigo Locatti