Question

I have stumbled on a compile error that makes no sense to me. So, of course, I turn to StackOverflow…..

System

Linux (Ubuntu 10.04 32 bit, gcc (Ubuntu 4.4.3-4ubuntu5.1) 4.4.3, Eclipse Indigo

** Question **

Obvious question, of course; What is gcc complaining about?

Background

I have a class, ThreadTime, coded in its own cpp and h files, ThreadTime.cpp and .h and inside its own namespace THREAD_TIME;

In main() I launch two threads, call then ThreadA and ThreadB, then go into an infinite loop that sleeps 2 seconds then checks to see if ThreadA and ThreadB are still running. in particular, if ThreadA has stopped, it gets restarted.

Each thread creates an associated ThreadTime object. In both of my threads there are loops. During each loop, the ThreadTimer get updated (in < 2 secs) so the main() loop can see that time is advancing in the threads.

As each ThreadTimer is created its address is added to a global vector, g_CthreadVector, of type std::vector<ThreadTime*>. g_CthreadVector is file global in ThreadType.cpp.

It’s probably best to just paste the whole class here. So…

ThreadTime.cpp

#include "ThreadTime.

namespace THREAD_TIME
{
   std::vector<ThreadTime*>      g_CthreadVector; // threads to kill and monitor

   ThreadTime::ThreadTime(thread_t* a, time_t b, void* c)
     : m_pthread(a), m_time(b), m_lasttime(b-2), m_function(c), m_terminate (FALSE)
   {
      m_name = "a ThreadTime object";
      g_CthreadVector.push_back( this );
   }

   ThreadTime::~ThreadTime()
   {
      std::vector<ThreadTime*>::iterator     it       (GetIterator());
      if ( it != g_CthreadVector.end() )
         g_CthreadVector.erase( it );
   }

   void     ThreadTime::UpdateTime( void )
   {
      m_lasttime = m_time;                                              // save the time to lasttime
      m_time = time(NULL);                                              // store the current time
      if ( m_lasttime == m_time )                                       // safety valve keep m_time
         m_lasttime--;                                                  //     ahead of m_lasttime
   }

   //---------------------------------------------------------
   // did time get advanced?
   //---------------------------------------------------------
   bool     ThreadTime::TimeAdvanced( void )
   {
      if ( m_lasttime < m_time )       return TRUE;
      else                             return FALSE;
   }

   void     ThreadTime::print( void )
   {
      std::vector<ThreadTime*>::iterator      tt_it   (GetIterator());
      std::vector<ThreadTime*>::iterator      it      (g_CthreadVector.end());

         for ( it = g_CthreadVector.begin(); it < g_CthreadVector.end(); it++ )
         {
            if ( tt_it == it )
               std::cout << "  ################### CURRENT OBJECT ###################### " << std::endl;
            else
               std::cout << "  $$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$ " << std::endl;

            std::cout << "  ThreadTime Object: " << (*it)->getName()    << std::endl
                      << "         thread ptr: " << (*it)->m_pthread    << std::endl
                      << "               time: " << (*it)->m_time       << std::endl
                      << "           lasttime: " << (*it)->m_lasttime   << std::endl
                      << "       function ptr: " << (*it)->m_function   << std::endl
                      << "          terminate: " << (*it)->m_terminate  << std::endl;
         }
         std::cout << "  $$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$ " << std::endl;
   }     // for ( it ... )

   std::vector<ThreadTime*>::iterator  ThreadTime::GetIterator( void )
   {
      std::vector<ThreadTime*>::iterator      it      (g_CthreadVector.end());

      for ( it = g_CthreadVector.begin(); it < g_CthreadVector.end(); it++)
      {
         if ( (*it)->m_pthread == m_pthread )
            break;
      }

      return it;
   }

} /* namespace THREAD_TIME */

ThreadTime.h

#ifndef THREADTIME_H_
   #define THREADTIME_H_

   #include "platform.h"
   #include "opsys_common.h"
   #include "NAS_Thread.h"


   namespace THREAD_TIME
   {
      class ThreadTime;

      class ThreadTime
      {
         public:
            std::string       getName() const                  { return m_name; }
            void              setName(std::string name)        { m_name = name; }

            ThreadTime(thread_t* a, time_t b, void* c);
            ~ThreadTime();
            void     UpdateTime( void );
            bool     TimeAdvanced( void );
            void     print( void );

         private:
            std::vector<ThreadTime*>::iterator       GetIterator( void );

            std::string    m_name;
            thread_t*      m_pthread;
            time_t         m_time;
            time_t         m_lasttime;
            void*          m_function;
            bool           m_terminate;
   };

} /* namespace THREAD_TIME */

#endif /* THREADTIME_H_ */

And finally, here are the error messages:

expected ‘;’ before ‘<’ token ThreadTime.h line 37

invalid use of ‘::’ ThreadTime.h Line 37

ISO C++ forbids declaration of ‘vector’ with no type ThreadTime.h line 37

Answer 1

There is no declaration of std::vector in scope for your class. You need a #include <vector>.