changeset 2589:b7e507e31613

PR1340: Simplify the rhino class rewriter to avoid use of concurrency 2013-03-11 Andrew John Hughes <gnu.andrew@member.fsf.org> PR1340: Simplify the rewriter, avoiding concurrency. * NEWS: Mention fix. * rewriter/com/redhat/rewriter/ClassRewriter.java: Always use the single threaded executor.
author Andrew John Hughes <gnu_andrew@member.fsf.org>
date Mon, 11 Mar 2013 16:31:39 +0000
parents 03aaabc8b71c
children f6cbaa56daa3
files ChangeLog NEWS rewriter/com/redhat/rewriter/ClassRewriter.java
diffstat 3 files changed, 11 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Mon Mar 11 11:39:51 2013 +0000
+++ b/ChangeLog	Mon Mar 11 16:31:39 2013 +0000
@@ -1,3 +1,10 @@
+2013-03-11  Andrew John Hughes  <gnu.andrew@member.fsf.org>
+
+	PR1340: Simplify the rewriter, avoiding concurrency.
+	* NEWS: Mention fix.
+	* rewriter/com/redhat/rewriter/ClassRewriter.java:
+	Always use the single threaded executor.
+
 2013-03-11  Andrew John Hughes  <gnu.andrew@member.fsf.org>
 
 	* Makefile.am:
--- a/NEWS	Mon Mar 11 11:39:51 2013 +0000
+++ b/NEWS	Mon Mar 11 16:31:39 2013 +0000
@@ -22,6 +22,7 @@
   - S8006882: Proxy generated classes in sun.proxy package breaks JMockit
 * Bug fixes
   - PR1303: Correct #ifdef to #if
+  - PR1340: Simplify the rhino class rewriter to avoid use of concurrency
 
 New in release 2.2.6 (2013-02-20):
 
--- a/rewriter/com/redhat/rewriter/ClassRewriter.java	Mon Mar 11 11:39:51 2013 +0000
+++ b/rewriter/com/redhat/rewriter/ClassRewriter.java	Mon Mar 11 16:31:39 2013 +0000
@@ -60,8 +60,8 @@
   /**
    * The executor for submitting rewriting jobs.
    */
-  private static final ExecutorService executor = DEBUG ?
-    Executors.newSingleThreadExecutor() : Executors.newCachedThreadPool();
+  private static final ExecutorService executor = 
+    Executors.newSingleThreadExecutor();
 
   /**
    * The source directory, set once by main.
@@ -290,4 +290,4 @@
     return null;
   }
 
-}
\ No newline at end of file
+}