|
All the articles I have read state that all threads must be terminated before the main thread exits or the program will still be resident in memory.
The question then becomes, how to properly terminate all worker threads?
I have the following:
void CClient::StopThread()<br />
{<br />
m_bRunning = false;<br />
WaitForSingleObject(m_pThread->m_hThread, INFINITE);<br />
<br />
delete m_pThread;<br />
}
And the worker thread is defined as:
UINT CClient::WorkerThread() <br />
{<br />
<br />
while(m_bRunning)<br />
{<br />
}<br />
<br />
AfxEndThread(0,0);<br />
<br />
return 0;<br />
}
The problem is that the program hangs after I call the StopThread function. I am sure I need to signal the function that my thread has terminated but, since I am new to the MFC and multithreaded programming, I am not sure where to do that.
Any help is appreciated. Thanks.
|
|
|
|
|
Never call AfxEndThread, or any similar function. It's bound create problems.
1.
What happens here is that the compiler optimizes away the m_bRunning in your thread since the value is not changed inside the while statement.
Fix this by declaring the m_bRunning variable as 'volatile'.
2.
I assume you start your thread with a call to AfxBeginThread.
If so, you should create it suspended and set the CWinThread::m_bAutoDelete to false to be able to wait for the thread to finish. Delete the CWinThread pointer that you got from AfxBeginThread in your StopThread function.
If you don't do this, a race condition will occur that may result in the thread handle you are waiting for is invalid.
Hope this helps
--
Roger
It's supposed to be hard, otherwise anybody could do it!
|
|
|
|
|
|
masnu wrote: all threads must be terminated before the main thread exits or the program will still be resident in memory
This isn't true - In fact if the "main" thread exits the process exits and any other threads that may be executing are killed on the spot. This is why when exiting the main thread typically waits for any worker threads to finish (using WaitForsingleObject or friends) before bailing: this gives the wokers a chance to clean up after themselves (for example, delete temp files).
Steve
PS: As for how to properly terminate a thread, the correct answer is ask the thread to exit and then wait for it to do so.
|
|
|
|
|
Where can I get a complete list of Win32 error codes. In VC++5.0, am getting a Win32 error 3, the authors say its because of the path specified not found, searched the whole net, but still can't make out what exactly is causing this error...
Would be of great help...
Adnan
|
|
|
|
|
|
True, the System Error codes provide a description, but its the same generic description. Could you tell me what might be causing this runtime error? am specifying all the paths of the DLL,exe,LIB. Am i missing any other environment settings...any help on that?
|
|
|
|
|
Spykraft wrote: Could you tell me what might be causing this runtime error?
Not without seeing the line of code in question.
Spykraft wrote: ...am specifying all the paths of the DLL,exe,LIB.
But are they the correct paths? Are they absolute or relative?
"The greatest good you can do for another is not just to share your riches but to reveal to him his own." - Benjamin Disraeli
|
|
|
|
|
DavidCrow wrote: But are they the correct paths? Are they absolute or relative?
AM giving absolute paths of the DLL and the .exe of my project in the Tools-options-directories and also filling up the System PATH vairable with the same paths, though am still getting that error.
do you have an idea, what are the general locations the loader looks for while executing an .exe ?
Imagine am working on a weekend!
|
|
|
|
|
I usually try to set the error code to display errors when this happens.
SetErrorMode(0) so that all errors are displayed. When all errors are displayed, Windows will tell you which DLL is missing or exported function is not found that your application is trying to load or find. Usually, an application has the error mode set to something other than 0 because it is 'handling' the errors.
Try that and see if you get more information from your program.
People that start writing code immediately are programmers (or hackers), people that ask questions first are Software Engineers - Graham Shanks
|
|
|
|
|
Look for winerror.h (in VC6, open a new text document, type winerror.h, select it, right-click on it and select "Open document") for most error codes you will ever encounter. Path not found is exactly that - like trying to open C:\BadDir\MoreBadDir\File.txt when the BadDir directory does not exist.
Peace!
-=- James
If you think it costs a lot to do it right, just wait until you find out how much it costs to do it wrong!
Avoid driving a vehicle taller than you and remember that Professional Driver on Closed Course does not mean your Dumb Ass on a Public Road!
DeleteFXPFiles & CheckFavorites (Please rate this post!)
|
|
|
|
|
|
Michael Dunn wrote: Use the Error Lookup tool in VC to get text descriptions of error codes.
You think I wudn've done that already mate?!
|
|
|
|
|
mukherjee124 wrote: You think I wudn've done that already mate?!
Oh btw, the replies by the name of Mukherjee123 are actually by me...someone forgot to sign outta of codeproject the last time!!!....
so all the replies posted recently--
mujherjee124 = SpyKraft
|
|
|
|
|
Could any body tell me how to hide/disable the horizontal and vertical scrollbar of CListCtrl? Thanks in advance.
|
|
|
|
|
I just did a test for you:
YourList.EnableScrollBarCtrl(SB_VERT,0);
A nice tool for optimizing your Microsoft html-help contents.
Includeh10
|
|
|
|
|
Hi there.
Lets say that I want to see wich way of output is faster: "printf" or "cout". I would writte something like this:
#include <stdio.h>
#include <time.h>
int main( void )
{
time_t begin, end;
begin = time( NULL );
unsigned long i;
for ( i=0 ; i<4294967295 ; i++ )
printf( "%d\n", i ); // This is for printf. If I was testing cout it would be
// cout << i << endl;
end = time( NULL );
printf( "IT TOOK: %d\n", end - begin );
return 0;
}
My intention is not to test wich of those is faster but to have an algorithm to test the speed for any application. The problem with this implementation is that it is not very precise; computers work a lot faster than what can be measured with seconds. Any suggestions?
Regards.
hint_54
|
|
|
|
|
hint_54 wrote: wich way of output is faster: "printf" or "cout".
fprintf() certainly.
but the 2 ones are no way comparable.
cout is 100% OO...
you could however test using GetTickCounter()... this is the best way to know.
TOXCCT >>> GEII power
[toxcct][VisualCalc 2.20][VCalc 3.0 soon...]
|
|
|
|
|
That was it 
Thanks
hint_54
|
|
|
|
|
Actually I'm not getting the results I was especting with this code.
int main( void )
{
unsigned long Begin, End, i;
tVar<int> *tVariant;
// Marks app init
Begin = GetTickCount();
// Writte testing stuff here
for ( i=0 ; i<500 ; i++ ) { // 4294967295
tVariant = new tVar<int>;
tVariant->TContextData = i;
delete tVariant;
}
// Marks app end
End = GetTickCount();
printf( "\nIT TOOK: %ld\n", End - Begin );
return 0;
}
I get an output of: "IT TOOL: 0". Know why!? 
regards
hint_54
|
|
|
|
|
hint_54 wrote: I get an output of: "IT TOOL: 0". Know why!?
Because the loop is only executing 500 times.
"The greatest good you can do for another is not just to share your riches but to reveal to him his own." - Benjamin Disraeli
|
|
|
|
|
Isnt there some other function more precise than this one?
regards
hint_54
|
|
|
|
|
Not when what you are wanting to measure happens in just a few ms. You can increase the number of times the loop executes (e.g., 250,000) to verify that it is being timed.
To get a little more granularity, you can use the multimedia timers.
Also check out QueryPerformanceCounter() and its API.
Because Windows is not a RTOS, you'll do no better than about 10ms for a Windows NT-based machine.
"The greatest good you can do for another is not just to share your riches but to reveal to him his own." - Benjamin Disraeli
|
|
|
|
|
hint_54 wrote: I get an output of: "IT TOOL: 0". Know why!?
If you are building it in Release mode, I bet the compiler simply optimized away your loop.
My programming blahblahblah blog. If you ever find anything useful here, please let me know to remove it.
|
|
|
|
|
No. _Debug.
regards
hint_54
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
