🚀 go-pugleaf

Thanks,

I wish I knew about that earlier...at least I've learnt something from all
of this.

"Mark Guerrieri" <mguerrieri@nospam_cmsgrp.com> wrote in message
news:424c8049$1@newsgroups.borland.com...
> 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
> >
> >
>
>