Comment 31 for bug 255149

details: http://icedtea.classpath.org//people/andrew/icedtea8?cmd=changeset;node=8dc6e1ff8ccf
author: Andrew John Hughes <email address hidden>
date: Thu May 20 14:37:32 2010 +0100

 Fix a number of issues and enable the Rhino rewriter in the build.

 2010-02-17 Andrew John Hughes <email address hidden>

  PR icedtea/179
  * ChangeLog: Remove rogue whitespace highlighted
  by emacs.
  * Makefile.am:
  (REWRITER_SRCS): Sources for class file rewriter.
  Only set RHINO_JAR when WITH_RHINO is set. Point
  to rewritten JAR file in build directory. Add
  rewriter and license to EXTRA_DIST. Make icedtea,
  icedtea-debug and icedtea-boot depend on rewrite-rhino.stamp.
  (stamps/rewriter.stamp): Build the class file rewriter.
  (stamps/rewrite-rhino.stamp): Rewrite the system Rhino JAR
  file to use the sun.org.mozilla namespace to avoid conflicts.
  (rewriter): New alias for stamps/rewriter.stamp.
  (rewrite-rhino): Likewise for stamps/rewrite-rhino.stamp.
  * patches/icedtea-rhino.patch:
  Copy rather than symlink the JAR file. Only drop the
  internal suffix on the javascript package names.
  * rewriter/com/redhat/rewriter/ClassRewriter.java:
  (executor): Use a single threaded executor when
  debugging is enabled.
  (tasks): Store the Futures returned by the executor
  for later checking.
  (main(String[])): Log what happens when processFile
  returns and handle any exceptions using Futures.
  (call()): Ensure srcDir/destDir replacement always
  matches with a trailing slash to avoid odd rewrites.
  Loop directory creation to avoid possible race issues.
  Increase logging and fix a number of issues with the rewrite:
    * Fix off-by-one loop bug so final entry is inspected.
    * Handle double entries which occur with 8-byte entries
      (doubles and longs):
  http://java.sun.com/docs/books/jvms/second_edition/html/ClassFile.doc.html#16628