All exceptions defined in khmer are derived class from khmer::khmer_e…

…xception, including khmer::khmer_file_exception.
dib-lab · Jul 22, 2014 · 92135e8 · 92135e8
1 parent 8d6b1cc
commit 92135e8
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 19 deletions.
diff --git a/lib/khmer.hh b/lib/khmer.hh
@@ -58,7 +58,7 @@ __attribute__((cpychecker_type_object_for_typedef(typename)))
 #   define MIN( a, b )	(((a) > (b)) ? (b) : (a))
 #   define MAX( a, b )	(((a) < (b)) ? (b) : (a))
 
-#include <khmer_exception.hh>
+#include "khmer_exception.hh"
 namespace khmer
 {
 // largest number we can count up to, exactly. (8 bytes)
@@ -79,7 +79,7 @@ typedef void (*CallbackFn)(const char * info, void * callback_data,
                            unsigned long long n_reads,
                            unsigned long long other);
 
-struct InvalidStreamBuffer : public std:: exception {
+struct InvalidStreamBuffer : public khmer_exception {
 };
 
 

diff --git a/lib/khmer_exception.hh b/lib/khmer_exception.hh
@@ -9,6 +9,7 @@
 #define KHMER_EXCEPTION_HH
 
 #include <exception>
+#include <string>
 
 namespace khmer
 {
@@ -21,19 +22,26 @@ namespace khmer
 class khmer_exception : public std::exception
 {
 public:
-    khmer_exception(const char * msg) : _msg(msg) { };
+    explicit khmer_exception(const char * msg) : _msg(msg) { }
+    explicit khmer_exception(const std::string& msg = "Generic khmer exception")
+        : _msg(msg.c_str()) { }
+
+    virtual ~khmer_exception() throw() { }
+    virtual const char* what() const throw () { return _msg; }
 
-    virtual const char* what() const throw() {
-        return _msg;
-    }
 protected:
     const char * _msg;
 };
 
+///
+// A base class for file exceptions.
+//
 class khmer_file_exception : public khmer_exception
 {
 public:
-    khmer_file_exception(const char * msg) : khmer_exception(msg) { };
+    explicit khmer_file_exception(const char * msg) : khmer_exception(msg) { }
+    explicit khmer_file_exception(const std::string& msg)
+        : khmer_exception(msg) { }
 };
 
 }

diff --git a/lib/perf_metrics.hh b/lib/perf_metrics.hh
@@ -12,16 +12,14 @@
 #include <cstring>
 #include <ctime>
 
-#include <exception>
 
 #include "khmer.hh"
 
-
 namespace khmer
 {
 
 #ifdef WITH_INTERNAL_METRICS
-struct InvalidPerformanceMetricsKey : public std:: exception {
+struct InvalidPerformanceMetricsKey : public khmer_exception {
 };
 
 

diff --git a/lib/read_parsers.hh b/lib/read_parsers.hh
@@ -84,22 +84,22 @@ public:
 
 };
 
-struct CacheSegmentUnavailable : public std:: exception {
+struct CacheSegmentUnavailable : public  khmer_exception {
 };
 
-struct CacheSegmentBoundaryViolation : public std:: exception {
+struct CacheSegmentBoundaryViolation : public  khmer_exception {
 };
 
-struct InvalidCacheSizeRequested : public std:: exception {
+struct InvalidCacheSizeRequested : public  khmer_exception {
 };
 
-struct NoMoreReadsAvailable : public std:: exception {
+struct NoMoreReadsAvailable : public  khmer_exception {
 };
 
-struct UnknownPairReadingMode : public std:: exception {
+struct UnknownPairReadingMode : public  khmer_exception {
 };
 
-struct InvalidReadPair : public std:: exception {
+struct InvalidReadPair : public  khmer_exception {
 };
 
 #ifdef WITH_INTERNAL_METRICS

diff --git a/lib/thread_id_map.hh b/lib/thread_id_map.hh
@@ -8,7 +8,6 @@
 #ifndef THREAD_ID_MAP_HH
 #   define THREAD_ID_MAP_HH
 
-#include <exception>
 #include <map>
 
 // TODO? Just use 'pthread_t' everywhere.
@@ -31,10 +30,10 @@ namespace khmer
 {
 
 
-struct InvalidNumberOfThreadsRequested : public std:: exception {
+struct InvalidNumberOfThreadsRequested : public khmer_exception {
 };
 
-struct TooManyThreads : public std:: exception {
+struct TooManyThreads : public khmer_exception {
 };