CODEWITHSUNDEEP
c++multithreadingpthreads
nihirus
nihirus

Reputation: 191

pthread_create argument passing error

Trying to write some simple multithreaded server program and got that error recently:

Server.cpp:64:64: error: argument of type ‘void* (Server::)(void*)’ does not match ‘void* (*)(void*)

Here is some lines from my code:

Header file:

class Server
{
public:
void establishConnection(const char * );
...

private:
void *listening(void *);
void *acceptingConnection(void *);
pthread_attr_t attrr;
}

cpp file:

void Server:: establishConnection(const char *port_number )
{
...
  pthread_create(&listn, &attrr, Server::listening, (void*)socketfd);//pthread_t listn, socketfd is a socket destricptor(int)     
  pthread_join(listn, NULL);

}
void* Server::listening(void *arg)
{
  int socketfd = (int)arg;
  ...
}

Normally, if I define thread function prototypes in the cpp file instead of header file, it works properly(without Server:: definition of course) Tried few other things like (void*)Server::listening, listening, (void*)listening but still didnt work. Could you enlighten me? How to pass the method parameter to listening method?

Secondly, I am learning c++ currently(already know C), is it true to use some C methods, char* arrays instead of strings, header files in the c++ program? Such as string.h, stdlib.h, pthread.h?

Upvotes: 0

Views: 945

Answers (2)

jxh
jxh

Reputation: 70502

You need to create a wrapper function for pthread_create(), and from there call into your class method.

class Server
{
...   
private:
int sock;
};

extern "C" void * server_listening (void *arg) {
    Server *s = static_cast<Server *>(arg);
    return s->listening();
}

void Server:: establishConnection(const char *port_number )
{
...
  this->sock = socketfd;
  pthread_create(&listn, &attrr, server_listening, this);
  pthread_join(listn, NULL);
}

The extern "C" linkage on the wrapper function is in place since pthread_create() is a C function, and expects a function pointer with C linkage. This is important if on your system the C ABI and the C++ ABI are not the same. A static method of a class can only have C++ linkage.

Upvotes: 1

user529758
user529758

Reputation:

You can just read the error message:

type ‘void* (Server::)(void*)’ does not match ‘void* (*)(void*)‘

Because Server::listening is a non-static member function of Server, and a pointer non-static member function cannot possibly be converted to a pointer-to-non-member-function.

You have to make your Server::listening function static, or write a stand-alone function outside the Server class.

Upvotes: 1

Related Questions