]> Creatis software - gdcm.git/commitdiff
Refine message displaying. Now, we have :
authorjpr <jpr>
Tue, 3 Jan 2006 14:28:53 +0000 (14:28 +0000)
committerjpr <jpr>
Tue, 3 Jan 2006 14:28:53 +0000 (14:28 +0000)
 * Warning  : Warns the user when some oddity occured, and we used an heuristics
 *            to bypass the trouble.
 *            e.g.  : SamplesPerPixel missing, we assume it's a grey level image
 *            e.g   : Pixel Representation missing : we assume 'unsigned'
 *            (we hope our assumption is OK, but user may disagree.)
 * Log      : we display messages when anything is not Dicom Kosher
 *            e.g. : non even length field
 *            e.g  : file is declared as Explicit VR, but a DataElement
 *                 is Implicit
 *            e.g  : a file holds an illegal group (0x0005, ...)
 * Debug : We help developper to trace, at a glance, the execution.
           (before refining with a debugging tool)
 * Setting ON Debug leads to set ON Warning (but not Log)
 * Setting ON Log   leads to set ON Warning (but not Debug)

src/gdcmDebug.cxx
src/gdcmDebug.h

index 31afc8e785eed1022428a0137242178cc17013fd..81c5c819f61207754ff265523b1f8632df048080 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDebug.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/11/28 16:50:32 $
-  Version:   $Revision: 1.29 $
+  Date:      $Date: 2006/01/03 14:28:53 $
+  Version:   $Revision: 1.30 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -28,6 +28,7 @@ namespace gdcm
 const int Debug::LINE_LENGTH = 79;
 
 bool Debug::DebugFlag     = false;
+bool Debug::LogFlag       = false;
 bool Debug::WarningFlag   = false;
 bool Debug::OutputToFile  = false;
 
@@ -66,12 +67,28 @@ void Debug::SetDebugFlag (bool flag)
  */ 
 void Debug::SetWarningFlag (bool flag) 
 {
-   // Cannot unset Warning flag if Debug flag is on.
-   if (flag == false && DebugFlag == true)
-      return;
+   // Cannot unset Warning flag if Debug flag is on or if LogFlag is on.
+   if (flag == false)
+   { 
+      if (DebugFlag == true)
+         return;
+      if (LogFlag == true)
+         return;
+   }
    WarningFlag = flag;
 }
 
+/**
+ * \brief   Sets the log flag
+ * @param   flag Set the log flag
+ */ 
+void Debug::SetLogFlag (bool flag) 
+{
+   // To log oddities, both flags are necessary      
+   WarningFlag = flag;   
+   LogFlag = flag;
+}
+
 /**
  * \brief   Accessor
  * @param   flag whether we want to redirect to file
index 172464871634499bc06e3b3e4c4b2b663e141641..4f5462c3f03b245a405c55a91f80f726653920ee 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDebug.h,v $
   Language:  C++
-  Date:      $Date: 2005/11/29 17:21:34 $
-  Version:   $Revision: 1.52 $
+  Date:      $Date: 2006/01/03 14:28:53 $
+  Version:   $Revision: 1.53 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -35,11 +35,13 @@ class CommandManager;
 
 //-----------------------------------------------------------------------------
 /**
- * \brief Debug is an object for debugging in program.
- * It has 2 debugging modes :
+ * \brief Debug is an object for warning/logging/tracing programs.
+ * It has the following modes :
  *  - error : for bad library use, seriously wrong DICOM
  *  - debug : for information/debug messages
- *  - warning : for warning about DICOM quality (kosher)
+ *  - warning : Warns the user when some oddity occured.
+ *  - log     : we display messages when anything is not Dicom Kosher
+ *  - debug : we help developper to trace, at a glance, the execution
  *  - assert : design by contract implementation. A function should have 
  *             proper input and proper output. 
  *             (should not happen, not user controlled)
@@ -47,7 +49,24 @@ class CommandManager;
  * A debugging message is only shown if the flag is on (DebugFlag)
  * This is static var and can be set at beginning of code:
  *         gdcm::Debug::SetDebugOn();
+ *
+ * Warning  : Warns the user when some oddity occured, and we used an heuristics
+ *            to bypass the trouble.
+ *            e.g.  : SamplesPerPixel missing, we assume it's a grey level image
+ *            e.g   : Pixel Representation missing : we assume 'unsigned'
+ *            (we hope our assumption is OK, but user may disagree.)
+ * Log      : we display messages when anything is not Dicom Kosher 
+ *            e.g. : non even length field
+ *            e.g  : file is declared as Explicit VR, but a DataElement
+ *                 is Implicit
+ *            e.g  : a file holds an illegal group (0x0005, ...)    
+ * Debug : We help developper to trace, at a glance, the execution.
+ *         (before refining with a debugging tool)
+ *
+ * Setting ON Debug leads to set ON Warning (but not Log)
+ * Setting ON Log   leads to set ON Warning (but not Debug)
  */
+
 class GDCM_EXPORT Debug
 {
 public:
@@ -65,6 +84,16 @@ public:
    /// \brief Sets the Debug Flag to false
    static void DebugOff () { SetDebugFlag(false); }
    
+   /// \brief This is a global flag that controls whether 
+   ///        log messages are displayed.
+   static void SetLogFlag (bool flag);
+   /// \brief   Gets the Log flag value
+   static bool GetLogFlag () {return LogFlag;}
+   /// \brief Sets the Log Flag to true
+   static void LogOn  () { SetLogFlag(true);  }
+   /// \brief Sets the Log Flag to false
+   static void LogOff () { SetLogFlag(false); } 
+   
    /// \brief This is a global flag that controls whether 
    ///        warning messages are displayed.
    static void SetWarningFlag (bool flag);
@@ -73,7 +102,7 @@ public:
    /// \brief Sets the Warning Flag to true
    static void WarningOn  () { SetWarningFlag(true);  }
    /// \brief Sets the Warning Flag to false
-   static void WarningOff () { SetWarningFlag(false); }   
+   static void WarningOff () { SetWarningFlag(false); }      
 
    /// \brief This is a global flag that controls if debug are redirected
    ///        to a file or not
@@ -92,8 +121,10 @@ public:
                             const Base *object = NULL);
 
 private:
-   static bool DebugFlag;
    static bool WarningFlag;
+   static bool LogFlag;
+   static bool DebugFlag;
+
    static bool OutputToFile;
 
    static std::ofstream OutputFileStream;
@@ -139,6 +170,8 @@ private:
    gdcm::Debug::SendToOutput(type,osmacro.str(),obj);          \
 }
 
+// ------------------------------------------------------------------------
+
 /**
  * \brief Debug : To be used to help bug tracking developer
  * @param msg message part
@@ -168,6 +201,26 @@ private:
    gdcmDebugBodyMacro(NULL,msg)
 #endif //NDEBUG
 
+// ------------------------------------------------------------------------
+
+/**
+ * \brief Log : we display messages when anything is not Dicom Kosher
+ * @param msg message part
+ */
+// No NDEBUG test to always have a return of warnings !!!
+// -> Rien compris! JPRx
+#define gdcmLogBodyMacro(obj, msg)                         \
+{                                                              \
+   if( Debug::GetLogFlag() )                               \
+      gdcmMessageBodyMacro(gdcm::CMD_LOG,obj,msg,"");      \
+}
+#define gdcmLogMacro(msg)                                  \
+   gdcmLogBodyMacro(this,msg)
+#define gdcmStaticLogMacro(msg)                            \
+   gdcmLogBodyMacro(NULL,msg)
+   
+// ------------------------------------------------------------------------
+
 /**
  * \brief Warning : To be used to warn the user when some oddity occurs
  * @param msg message part
@@ -184,6 +237,8 @@ private:
 #define gdcmStaticWarningMacro(msg)                            \
    gdcmWarningBodyMacro(NULL,msg)
 
+// ------------------------------------------------------------------------
+
 /**
  * \brief   Error : To be used when unecoverabale error occurs
  *          at a 'deep' level. (don't use it if file is not ACR/DICOM!)
@@ -200,6 +255,8 @@ private:
 #define gdcmStaticErrorMacro(msg)                              \
    gdcmErrorBodyMacro(NULL,msg)
 
+// ------------------------------------------------------------------------
+
 /**
  * \brief Assert : To be used when an *absolutely* impossible error occurs
  *        No function should be allowed to stop the process instead of