* cygthread.cc (cygthread::stub): Remove old, unnecessary, FIXMEd code.

* spawn.cc (child_info_spawn::worker): Avoid calling close_all_files() twice.

winsup/cygwin/ChangeLog

@@ -1,3 +1,9 @@
+2012-09-13  Christopher Faylor  <me.cygwin2012@cgf.cx>
+
+	* cygthread.cc (cygthread::stub): Remove old, unnecessary, FIXMEd code.
+	* spawn.cc (child_info_spawn::worker): Avoid calling close_all_files()
+	twice.
+
 2012-09-02  Jin-woo Ye <jojelino@gmail.com>
 	    Corinna Vinschen  <corinna@vinschen.de>
 

winsup/cygwin/cygthread.cc

@@ -90,12 +90,6 @@ cygthread::stub (VOID *arg)
 #endif
       else
 	{
-	  if (exiting)
-	    {
-	      info->inuse = false;	// FIXME: Do we need this?
-	      return 0;
-	    }
-
 	  info->callfunc (false);
 
 	  HANDLE notify = info->notify_detached;

winsup/cygwin/release/1.7.17

@@ -24,3 +24,7 @@ Bug fixes:
 
 - Prefer to call WSARecvMsg from recvmsg if possible.
   Fixes: http://cygwin.com/ml/cygwin/2012-07/msg00662.html
+
+- Correct hang typified by running "procps" after Cygwin has started a
+  Windows process.
+  Fixes: http://cygwin.com/ml/cygwin/2012-05/threads.html#00349

winsup/cygwin/spawn.cc

@@ -867,7 +867,8 @@ loop:
 	}
       else
 	{
-	  close_all_files (true);
+	  if (iscygwin ())
+	    close_all_files (true);
 	  if (!my_wr_proc_pipe
 	      && WaitForSingleObject (pi.hProcess, 0) == WAIT_TIMEOUT)
 	    wait_for_myself ();