Threads ahhhh

[polus]

Hi, Im trying to create a worker thread and have followed/read several guides. I get this error when trying to compile:


error C2665: 'AfxBeginThread' : none of the 2 overloads can convert parameter 1 from type 'void (void)'


I dont understand, there is a funtion that takes void, void:
void CFModAudio::PlayThread( void )

Please help.


void CFModAudio::Play()
{
running = TRUE;
AfxBeginThread( PlayThread, (LPVOID)this );
}

UINT CFModAudio::PlayThread( LPVOID pcAudio )
{
CFModAudio *cAudio = (CFModAudio *)pcAudio;
cAudio->PlayThread();
return 0;
}

void CFModAudio::PlayThread( void )
{
// Do the worker task...
}

[Siddhartha]

I dont understand, there is a funtion that takes void, void:
void CFModAudio::PlayThread( void )It actually does not.

Observe your code closely - it returns UINT!

UINT CFModAudio::PlayThread( LPVOID pcAudio ) ;)

[Siddhartha]

[ redirected ]

Regards,
Siddhartha

[polus]

Penny................Drop

Still gives me the error :( hmmm


msdn:
pfnThreadProc
Points to the controlling function for the worker thread. Cannot be NULL. This function must be declared as follows:
UINT MyControllingFunction( LPVOID pParam );

My Function

UINT PlayThread( LPVOID pcAudio );

Error

error C2665: 'AfxBeginThread' : none of the 2 overloads can convert parameter 1 from type 'unsigned int (void *)'

[Siddhartha]

Actually you have two PlayThread functions, one of which return UINT and another void (just noticed).

Is the thread function static?
Check this FAQ (http://www.codeguru.com/forum/showthread.php?t=312452) for details.

[polus]

hehe thats where I got it from :)

Just realised, no static! given that static is vital to the operation of the new thread, I shall just go and kick myself in the head!

Thanks :)

[Siddhartha]

You are welcome... ;)

BTW, the function that is static, returns an UINT and accepts a LPVOID is best suited to your task.