[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-common-dev
Subject:    [Common-dev] CN-Client: small change in file log observer
From:       "Eric Hyche" <ehyche () real ! com>
Date:       2007-05-25 20:09:33
Message-ID: 009901c79f08$9d79abe0$db68a8c0 () EHYCHED620
[Download RAW message or body]


This is now checked into HEAD and 150Cay.

=============================================
Eric Hyche (ehyche@real.com)
Technical Lead
RealNetworks, Inc.  

> -----Original Message-----
> From: Greg Wright [mailto:gwright@real.com] 
> Sent: Friday, May 25, 2007 3:46 PM
> To: ehyche@real.com
> Cc: common-dev@lists.helixcommunity.org
> Subject: Re: [Common-dev] CR-Client: small change in file log observer
> 
> Looks good.
> --greg.
> 
> 
> Eric Hyche wrote:
> > Description
> > --------------------------------------
> > In the file log observer, we load the log system
> > upon initialization. Currently, we first look in
> > the encoding SDK directory. If we don't find it,
> > then we look in the Plugins directory.
> > 
> > With this change, if HELIX_FEATURE_CLIENT is defined,
> > then we first look in the Plugins directory and then
> > if that doesn't work, then we look in the Encoding SDK
> > directory.
> > 
> > This allows if we have separate log system binaries for
> > the Producer SDK and Helix DNA client to co-exist.
> > 
> > Files Modified
> > --------------------------------------
> > common/log/logobserverfile/hxtlogobserver.cpp
> > 
> > Branches
> > --------------------------------------
> > HEAD and 150Cay
> > 
> > Index: hxtlogobserver.cpp
> > ===================================================================
> > RCS file: /cvsroot/common/log/logobserverfile/hxtlogobserver.cpp,v
> > retrieving revision 1.19
> > diff -u -w -r1.19 hxtlogobserver.cpp
> > --- hxtlogobserver.cpp  11 Dec 2006 04:48:21 -0000      1.19
> > +++ hxtlogobserver.cpp  25 May 2007 19:32:27 -0000
> > @@ -316,15 +316,24 @@
> >      // Get the GetBuilderInterface method address from the 
> log system DLL
> >      FPRMAGETLOGSYSTEMINTERFACE fpGetLogSystem = NULL;
> >      m_pDllAccess = new DLLAccess();
> > -    // Try the DLLTYPE_ENCSDK directory first
> > -    m_pDllAccess->open(strDllName, DLLTYPE_ENCSDK);
> >  #if defined(HELIX_FEATURE_CLIENT)
> > +    // If we are built with HELIX_FEATURE_CLIENT, then
> > +    // try the Plugins directory first and then if
> > +    // that doesn't work try the EncodingSDK directory
> > +    //
> > +    // Try the Plugins directory
> > +    m_pDllAccess->open(strDllName, DLLTYPE_PLUGIN);
> >      if (!m_pDllAccess->isOpen())
> >      {
> > -        // Try the DLLTYPE_PLUGIN directory next
> > -        m_pDllAccess->open(strDllName, DLLTYPE_PLUGIN);
> > +        // Try the encoding SDK directory
> > +        m_pDllAccess->open(strDllName, DLLTYPE_ENCSDK);
> >      }
> > +#else /* #if defined(HELIX_FEATURE_CLIENT) */
> > +    // If we are not built with HELIX_FEATURE_CLIENT,
> > +    // then only try the EncodingSDK directory
> > +    m_pDllAccess->open(strDllName, DLLTYPE_ENCSDK);
> >  #endif /* #if defined(HELIX_FEATURE_CLIENT) */
> > +
> >      if(m_pDllAccess->isOpen())
> >      {
> >  #if defined(HELIX_FEATURE_CLIENT)
> > 
> > 
> > =====================================
> > Eric Hyche, Technical Lead
> > RealNetworks, Inc.
> > ehyche@real.com
> > 
> > 
> > _______________________________________________
> > Common-dev mailing list
> > Common-dev@helixcommunity.org
> > http://lists.helixcommunity.org/mailman/listinfo/common-dev
> 


_______________________________________________
Common-dev mailing list
Common-dev@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/common-dev
[prev in list] [next in list] [prev in thread] [next in thread]