[kaffe] CVS kaffe (jim): Remove assertion that was breaking test/internal with pthreads

Kaffe CVS cvs-commits at kaffe.org
Mon Feb 16 20:17:01 PST 2004


PatchSet 4443 
Date: 2004/02/17 04:13:29
Author: jim
Branch: HEAD
Tag: (none) 
Log:
Remove assertion that was breaking test/internal with pthreads

Members: 
	ChangeLog:1.2026->1.2027 
	kaffe/kaffevm/systems/unix-pthreads/thread-impl.c:1.19->1.20 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.2026 kaffe/ChangeLog:1.2027
--- kaffe/ChangeLog:1.2026	Tue Feb 17 01:54:01 2004
+++ kaffe/ChangeLog	Tue Feb 17 04:13:29 2004
@@ -1,5 +1,11 @@
 2004-02-16  Jim Pick  <jim at kaffe.org>
 
+	* kaffe/kaffevm/systems/unix-pthreads/thread-impl.c:
+	Comment out assertion that was breaking test/internal
+	with pthreads.
+
+2004-02-16  Jim Pick  <jim at kaffe.org>
+
 	* configure.in:
 	Don't link to libasound if not using pthreads.  Also, fix
 	--disable-sound and --without-alsa options.
Index: kaffe/kaffe/kaffevm/systems/unix-pthreads/thread-impl.c
diff -u kaffe/kaffe/kaffevm/systems/unix-pthreads/thread-impl.c:1.19 kaffe/kaffe/kaffevm/systems/unix-pthreads/thread-impl.c:1.20
--- kaffe/kaffe/kaffevm/systems/unix-pthreads/thread-impl.c:1.19	Sun Feb  8 13:44:42 2004
+++ kaffe/kaffe/kaffevm/systems/unix-pthreads/thread-impl.c	Tue Feb 17 04:13:31 2004
@@ -474,7 +474,10 @@
   pthread_setcanceltype( PTHREAD_CANCEL_ASYNCHRONOUS, &oldCancelType);
 
   /* if we aren't the first one, we are in trouble */
-  assert( activeThreads == 0);
+  // Jim - I'm commenting out this assertion, as jit_stub.c in test/internal  
+  // seems to violate the assumption.
+  // assert( activeThreads == 0);
+ 
   activeThreads = firstThread = nt;
   nonDaemons=1;
   nSysThreads=1;




More information about the kaffe mailing list