my SLOT not part of QThread

[freak]

Hello, I got the following problem.

class mythread : public QThread
{
public:
     mythread();
     void run();
public slots:
     void processpacket();
 
private:
     QTcpServer *tcpServer;
};

class mythread : public QThread
{
public:
     mythread();
     void run();
public slots:
     void processpacket();

private:
     QTcpServer *tcpServer;
};

To copy to clipboard, switch view to plain text mode 

inside mythread::run I have this

connect(tcpServer, SIGNAL(newConnection()), this, SLOT(processpacket()));

connect(tcpServer, SIGNAL(newConnection()), this, SLOT(processpacket()));

To copy to clipboard, switch view to plain text mode 

everything compiles and builds but when I run it I get a

Object::connect: No such slot QThread::processpacket().

Object::connect: No such slot QThread::processpacket().

To copy to clipboard, switch view to plain text mode 

and if I add Q_OBJECT as follows:

class mythread : public Qthread
{
Q_OBJECT
 
public:
     mythread();
     void run();
public slots:
     void processpacket();
 
private:
     QTcpServer *tcpServer;
};

class mythread : public Qthread
{
Q_OBJECT

public:
     mythread();
     void run();
public slots:
     void processpacket();

private:
     QTcpServer *tcpServer;
};

To copy to clipboard, switch view to plain text mode 

I get the following error when compiling.:

undefined reference to 'vtable for mythread'

Any help in solving this issues is appreciated. Thanks in advanced

[jpn]

Re-run qmake after adding Q_OBJECT macro.

[freak]

Thanks Jpn, that solved both problems. Now i have a problem where the SLOT code is never executed. I send packets from my client machine but it does nothing. is the code shown above incorrect?

Thanks in advanced

[jacek]

connect(tcpServer, SIGNAL(newConnection()), this, SLOT(processpacket()));

Beware that a QThread object and all objects created in its constructor live in a thread where that QThread instance was created, while all objects created in QThread::run() live in a newly created thread.

This means that the above connection will be a queued connection and processpacket() won't be executed by the new thread. Either move those slots to some other object that you will create in QThread::run() or make that connection a direct one explicitly.

[jacek]

Now i have a problem where the SLOT code is never executed.

Do you start the event loop in that new thread (i.e. do you invoke QThread::exec() in run() )?

[freak]

I do call exec() inside run.

How do you make a connection explicitly?

[jacek]

How do you make a connection explicitly?

connect( tcpServer, SIGNAL( newConnection() ),
         this, SLOT( processpacket() ), 
         Qt::DirectConnection );

connect( tcpServer, SIGNAL( newConnection() ),
         this, SLOT( processpacket() ), 
         Qt::DirectConnection );

To copy to clipboard, switch view to plain text mode