[Common-dev] CR-Client: Minor logging system changes

[Common-dev] CR-Client: Minor logging system changes

Eric Hyche ehyche at real.com
Thu May 27 07:24:42 PDT 2004



Synopsis: Make HXLOGLx macros available on both client and
           producer profile builds

Overview: When we integrated the producer logging system into
           the client, we did not use the same front end access
           functions that the producer team uses, since we didn't
           want to pay the price of linking in DLLAccess for each
           DLL that wanted to use logging. The front-end functions
           in common/log/logutil/hxtlogutil.cpp/.h were differentiated
           by HELIX_FEATURE_STATICDLLACCESS: If HELIX_FEATURE_STATICDLLACCESS
           was defined, you could use the RSLOG/HXLOG() macros the
           Producer team is used to using. If you didn't have
           HELIX_FEATURE_STATICDLLACCESS, then you used the HXLOGLx() macros
           the client team came up with. HELIX_FEATURE_STATICDLLACCESS was
           then defined in the helix-producer-all-defines profile.

           However, it turns out that the producer needs to build
           some client modules as well, such as the client core and mp3
           file format. These modules have already incorporated the HXLOGLx()
           macros, and therefore since HELIX_FEATURE_STATICDLLACCESS was
           defined in helix-producer-all-defines, then these modules failed due
           to the HXLOGLx() macros not being defined.

           Therefore, we need to make the HXLOGLx() macros available regardless
           of whether HELIX_FEATURE_STATICDLLACCESS is defined. The 
RSLOG/HXLOG()
           macros will still only be available if HELIX_FEATURE_STATICDLLACCESS
           is defined.

Files Modified:
common/log/logutil/hxtlogutil.cpp   - Make HXLOGLx macros available
common/log/logutil/pub/hxtlogutil.h - Make HXLOGLx macros available
datatype/mp3/fileformat/mp3ff.cpp   - Define ENABLE_DLLACCESS_PATHS 
conditionally
     on HELIX_FEATURE_STATICDLLACCESS.

Image Size and Heap Use impact: none

Platforms and Profiles Affected: those that define 
HELIX_FEATURE_STATICDLLACCESS (currently
          only producer profiles)

Distribution Libraries affected: none

Distribution library impact and planned action: n/a

Platforms and Profiles Build Verified: win32

Platforms and Profiles Functionality verified: win32

Branch: HEAD only

QA Instructions: none


Index: hxtlogutil.cpp
===================================================================
RCS file: /cvsroot/common/log/logutil/hxtlogutil.cpp,v
retrieving revision 1.2
diff -u -w -u -w -r1.2 hxtlogutil.cpp
--- hxtlogutil.cpp      29 Apr 2004 15:42:57 -0000      1.2
+++ hxtlogutil.cpp      27 May 2004 14:13:40 -0000
@@ -47,10 +47,9 @@
  static DLLAccess g_LogDLL;
  BOOL             g_bTriedInit = FALSE;
  CHXTMutex        g_CreationMutex;
-#else /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */
+#endif /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */
  #include "hlxclib/string.h"
  IHXDllAccessPtr          g_spDLLAccess;
-#endif /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */
  IHXTInternalLogWriterPtr g_spLog;
  IHXTLogSystemContextPtr  g_spLogSystemContext;
  IHXTLogSystemPtr         g_spLogSystem;
@@ -282,7 +281,7 @@
      }
  }

-#else /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */
+#endif /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */

  // These functions require HXEnableLogging() to be called
  // first before they will work
@@ -450,4 +449,3 @@
      }
  }

-#endif /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */
Index: pub/hxtlogutil.h
===================================================================
RCS file: /cvsroot/common/log/logutil/pub/hxtlogutil.h,v
retrieving revision 1.1.1.1
diff -u -w -u -w -r1.1.1.1 hxtlogutil.h
--- pub/hxtlogutil.h    28 Apr 2004 20:26:11 -0000      1.1.1.1
+++ pub/hxtlogutil.h    27 May 2004 14:13:40 -0000
@@ -87,7 +87,7 @@
  #define HXTLOG_DEBUG if(0)   // compile out
  #endif /* #if defined(_DEBUG) */

-#else /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */
+#endif /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */

  #include "hxdllaccess.h"
  #include "hxccf.h"
@@ -196,7 +196,5 @@

  // This should be kept to the max DLL suffix length
  #define HXLOG_MAXDLLSUFFIXLEN 7
-
-#endif /* #if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) */

  #endif /* #ifndef HXTLOGUTIL_H */
Index: mp3ff.cpp
===================================================================
RCS file: /cvsroot/datatype/mp3/fileformat/mp3ff.cpp,v
retrieving revision 1.33
diff -u -w -u -w -r1.33 mp3ff.cpp
--- mp3ff.cpp   29 Apr 2004 15:43:43 -0000      1.33
+++ mp3ff.cpp   27 May 2004 14:13:57 -0000
@@ -86,7 +86,7 @@

  //#include "rmfftype.h" // for the HX_SAVE_ENABLED flag

-#ifdef _AIX
+#if defined(HELIX_FEATURE_LOG_STATICDLLACCESS) || defined(_AIX)
  #include "dllpath.h"
  ENABLE_MULTILOAD_DLLACCESS_PATHS(Rnmp3fmt);
  #endif





======================================
M. Eric Hyche (ehyche at real.com)
Core Technologies
RealNetworks, Inc.




More information about the Common-dev mailing list
 

Site Map   |   Terms of Use   |   Privacy Policy   |   Contact Us

Copyright © 1995-2007 RealNetworks, Inc. All rights reserved. RealNetworks and Helix are trademarks of RealNetworks.
All other trademarks or registered trademarks are the property of their respective holders.