modifications to prepare for a proper error management. Changed the way OpenJPH version is created, and modified CMakeLists.txt to remove -g and add -DNDEBUG

Author: aous72

CMakeLists.txt

@@ -5,7 +5,7 @@ project (openjph VERSION 0.1.0 DESCRIPTION "Open source implementation of JPH" L
 if (MSVC)
 	set(CMAKE_CXX_FLAGS "/EHsc")
 else()
-	set(CMAKE_CXX_FLAGS "-std=c++11 -O3 -g")
+	set(CMAKE_CXX_FLAGS "-std=c++11 -O3 -DNDEBUG")
 endif()
 
 set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/../bin)

src/apps/apps.xcodeproj/project.xcworkspace/xcuserdata/aous.xcuserdatad/UserInterfaceState.xcuserstate

@@ -45,6 +45,7 @@
 #include "ojph_file.h"
 #include "ojph_codestream.h"
 #include "ojph_params.h"
+#include "ojph_message.h"
 
 /////////////////////////////////////////////////////////////////////////////
 struct size_list_interpreter : public ojph::cli_interpreter::arg_inter_base
@@ -728,6 +729,13 @@ int main(int argc, char * argv[]) {
     printf("%s\n", e);
     exit (-1);
   }
+  catch (const std::exception& e)
+  {
+    const char *p = e.what();
+    if (strncmp(p, "ojph error", 10) != 0)
+      printf("%s\n", p);
+    exit(-1);
+  }
 
   clock_t end = clock();
   double elapsed_secs = double(end - begin) / CLOCKS_PER_SEC;

src/apps/ojph_compress/ojph_compress.cpp

@@ -376,7 +376,10 @@ namespace ojph {
       if (!qcd.write(file))
         throw "Error writing to file";
 
-      char buf[] = "    OpenJPH Ver 0.1.1";
+      char buf[] = "    OpenJPH Ver "
+        OJPH_INT_TO_STRING(OJPH_CORE_VER_MAJOR) "."
+        OJPH_INT_TO_STRING(OJPH_CORE_VER_MINOR) "."
+        OJPH_INT_TO_STRING(OJPH_CORE_VER_SUBMINOR) ".";
       size_t len = strlen(buf);
       *(ui16*)buf = swap_byte(JP2K_MARKER::COM);
       *(ui16*)(buf + 2) = swap_byte((ui16)(len - 2));

src/core/codestream/ojph_codestream.cpp

@@ -154,10 +154,21 @@ namespace ojph {
     return reinterpret_cast<T *>(p);
   }
 
+  ////////////////////////////////////////////////////////////////////////////
+  //                         OS detection definitions
+  ////////////////////////////////////////////////////////////////////////////
+#if (defined WIN32) || (defined _WIN32) || (defined _WIN64)
+  #define OJPH_OS_WINDOWS
+#elif (defined __APPLE__)
+  #define OJPH_OS_APPLE
+#elif (defined __linux)
+  #define OJPH_OS_LINUX
+#endif
+
   /////////////////////////////////////////////////////////////////////////////
   // defines for dll
   /////////////////////////////////////////////////////////////////////////////
-#ifdef OJPH_COMPILER_MSVC
+#ifdef OJPH_OS_WINDOWS
   #define OJPH_EXPORT __declspec(dllexport)
 #else
   #define OJPH_EXPORT

src/core/common/ojph_arch.h

@@ -55,6 +55,14 @@ typedef int32_t si32;
 typedef uint64_t ui64;
 typedef int64_t si64;
 
+/////////////////////////////////////////////////////////////////////////////
+#define OJPH_CORE_VER_MAJOR    0
+#define OJPH_CORE_VER_MINOR    2
+#define OJPH_CORE_VER_SUBMINOR 0
+
+/////////////////////////////////////////////////////////////////////////////
+#define OJPH_INT_TO_STRING(I) #I
+
 /////////////////////////////////////////////////////////////////////////////
 // number of fractional bits for 16 bit representation
 // for 32 bits, it is NUM_FRAC_BITS + 16

src/core/common/ojph_defs.h

@@ -40,15 +40,41 @@
 
 namespace ojph {
 
-  ///////////////////////////////////////////////////////////////////////////
+  ////////////////////////////////////////////////////////////////////////////
   void set_error_stream(FILE *s);
 
-  ///////////////////////////////////////////////////////////////////////////
-  void set_output_stream(FILE *s);
+  ////////////////////////////////////////////////////////////////////////////
+  void set_warning_stream(FILE *s);
 
-  ///////////////////////////////////////////////////////////////////////////
-  void error(int error_code, const char* file_name, int line_num);
+  ////////////////////////////////////////////////////////////////////////////
+  void warn(int warn_code, const char* file_name, int line_num,
+            const char *fmt, ...)
+  #ifdef OJPH_COMPILER_GNUC
+    __attribute__((format(printf, 4, 5)))
+  #endif
+    ;
 
+  ////////////////////////////////////////////////////////////////////////////
+  void error(int error_code, const char* file_name, int line_num,
+             const char *fmt, ...)
+  #ifdef OJPH_COMPILER_GNUC
+    __attribute__((format(printf, 4, 5)))
+  #endif
+    ;
 }
 
+//////////////////////////////////////////////////////////////////////////////
+#if (defined OJPH_OS_WINDOWS)
+  #define __OJPHFILE__ \
+    (strrchr(__FILE__, '\\') ? strrchr(__FILE__, '\\') + 1 : __FILE__)
+#else
+  #define __OJPHFILE__ \
+    (strrchr(__FILE__, '/') ? strrchr(__FILE__, '/') + 1 : __FILE__)
+#endif
+//////////////////////////////////////////////////////////////////////////////
+#define OJPH_WARN(t, ...) ojph::warn(t, __OJPHFILE__, __LINE__, __VA_ARGS__);
+//////////////////////////////////////////////////////////////////////////////
+#define OJPH_ERROR(t, ...) ojph::error(t, __OJPHFILE__, __LINE__,__VA_ARGS__);
+
+
 #endif // !OJPH_MESSAGE_H

src/core/common/ojph_message.h

@@ -36,13 +36,15 @@
 /****************************************************************************/
 
 #include <cstdio>
+#include <cstdarg>
+#include <stdexcept>
 
 #include "ojph_message.h"
 
 namespace ojph {
 
   ////////////////////////////////////////////////////////////////////////////
-  FILE *output_stream = stdout;
+  FILE *warning_stream = stdout;
   FILE *error_stream = stderr;
 
   ////////////////////////////////////////////////////////////////////////////
@@ -52,21 +54,36 @@ namespace ojph {
   }
 
   ////////////////////////////////////////////////////////////////////////////
-  void set_output_stream(FILE *s)
+  void set_warning_stream(FILE *s)
   {
-    output_stream = s;
+    warning_stream = s;
   }
 
   ////////////////////////////////////////////////////////////////////////////
-  void error(int error_code, const char* file_name, int line_num)
+  void error(int error_code, const char* file_name, int line_num,
+             const char *fmt, ...)
   {
-    fprintf(error_stream, "cuda error: %d %s at %s:%d\n",
-      error_code, " ", file_name, line_num);
+    fprintf(error_stream, "ojph error %d at %s:%d\n",
+      error_code, file_name, line_num);
+    va_list args;
+    va_start(args, fmt);
+    fprintf(error_stream, fmt, args);
+    va_end(args);
 
-    throw;
+    throw std::runtime_error("ojph error");
   }
 
-
+  ////////////////////////////////////////////////////////////////////////////
+  void warn(int warn_code, const char* file_name, int line_num,
+            const char *fmt, ...)
+  {
+    fprintf(warning_stream, "ojph warning %d at %s:%d\n",
+      warn_code, file_name, line_num);
+    va_list args;
+    va_start(args, fmt);
+    fprintf(warning_stream, fmt, args);
+    va_end(args);
+  }
 
 
 }