🚀 go-pugleaf

FYI- have a look at TCriticalSection (or TCOMCriticalSection) for a pre-made
critical section class...

Mark

"Daniel Stevenson" <d.stevenson@ocreage.com.au> wrote in message
news:42408739$1@newsgroups.borland.com...
> Thank you very much, I thought I needed an extern declaration which I
tried
> but i wasn't declaring the object in the cpp file correctly.
>
> Thanks heaps, its all working now.
>
> "Eelke" <e.klein@mplussoftware.nl> wrote in message
> news:423fc937$1@newsgroups.borland.com...
> >
> > > I don't think there is anything wrong with the above declaration, I
> included
> > > this to reduce the number of questions in this area.
> > > I think the problem is in how I've declared my queue but I'm having a
> brain
> > > freeze and can't think straight to work out my problem as seen below.
> > >
> > >
> > > ....LoggingServiceSource.h...
> > >
>
//--------------------------------------------------------------------------
> > > -
> > > #ifndef LoggingServiceSourceH
> > > #define LoggingServiceSourceH
> > >
>
//--------------------------------------------------------------------------
> > > -
> > > #include <SysUtils.hpp>
> > > #include <Classes.hpp>
> > > #include <SvcMgr.hpp>
> > > #include <vcl.h>
> > > #include "thdLoggingSource.h"
> > > #include <OleServer.hpp>
> > > #include <queue>
> > > using namespace std;
> > >
>
//--------------------------------------------------------------------------
> > > -
> > > TLogQueue<LogStruct> LogQueue;            //declaring my queue
> > >
>
//--------------------------------------------------------------------------
> > > -
> > > class TLoggingService : public TService
> > > {
> > > __published:    // IDE-managed Components
> > >   TatSystemStatus *atSystemStatus;
> > >   TatPagingSubscription *atPagingSubscription;
> > >   void __fastcall atSystemStatusSystemStatus(TObject *Sender,
> > >           atSystemStatusType StatusType, BSTR Origin,
> > >           atSystemStatusLevel Level, BSTR Description);
> > >   void __fastcall atPagingSubscriptionPaging(TObject *Sender,
> > >           BSTR CallNumber, BSTR DisplayMessage, long Priority,
> > >           PagingInfoStruct *AdditionalInfo);
> > > private:        // User declarations
> > >   TLoggingThread *LoggingThread;
> > >   String ConnectionStatus;
> > >
> > >   void __fastcall UpdateServiceDescription(TService *Sender);
> > >   void __fastcall CreateWorkerThread();
> > >   void __fastcall CheckSystemStatus();
> > > public:         // User declarations
> > >  __fastcall TLoggingService(TComponent* Owner);
> > >  TServiceController __fastcall GetServiceController(void);
> > >
> > >  friend void __stdcall ServiceController(unsigned CtrlCode);
> > > };
> > >
>
//--------------------------------------------------------------------------
> > > -
> > > extern PACKAGE TLoggingService *LoggingService;
> > >
>
//--------------------------------------------------------------------------
> > > -
> > > #endif
> > >
> > You declared the queue in the header which causes each cpp which
> > includes this header to get its own copy. Change the declaration to
> >
> > extern TLogQueue<LogStruct> LogQueue;
> >
> > and put
> >
> > TLogQueue<LogStruct> LogQueue;
> >
> > in one cpp.
> >
> > Eelke
>
>