reSIProcate

Mailing Lists

Source Code

Google Search:



< Previous by Date Date Index Next by Date >
< Previous in Thread Thread Index Next in Thread >

Re: [reSIProcate-users] ThreadIf

If you're wondering what the problem was I have absolutely no idea. It was 8 years ago :-)

Sent from my iPhone

On Feb 24, 2012, at 8:52 AM, Scott Godin <sgodin@xxxxxxxxxxxxxxx> wrote:

That used to be code - however in revision 3083 - Jason F checked in this change with the following comment:

rev3083 - jason  09/07/2004 12:35:20 AM
fixed a bastard of problem with ThreadIf::join on windows. Threads should join correctly now. 

I'm not really sure in which cases the single code line wasn't working.  However, given that this change was made explicitly and I don't believe it is causing any problems, I see no reason to change it.

Scott

On Fri, Feb 24, 2012 at 8:12 AM, Karlsson <boost.regex@xxxxxxxxx> wrote:
In the threadIf.cxx, I think below code is not necessary:    

DWORD exitCode;
   while (true)
   {
      if (GetExitCodeThread(mThread,&exitCode) != 0)
      {
         if (exitCode != STILL_ACTIVE)
         {
            break;
         }
         else
         {
            WaitForSingleObject(mThread,INFINITE);
         }
      }
      else
      {
         // log something here
         break;
      }
   }



This is enough:

WaitForSingleObject(mThread,INFINITE);

 

_______________________________________________
resiprocate-users mailing list
resiprocate-users@xxxxxxxxxxxxxxx
List Archive: http://list.resiprocate.org/archive/resiprocate-users/

_______________________________________________
resiprocate-users mailing list
resiprocate-users@xxxxxxxxxxxxxxx
List Archive: http://list.resiprocate.org/archive/resiprocate-users/