[Bug classpath/67189] New: [PATCH] CharsetEncoder changes to support Armed Bear Common Lisp (ABCL)

[wispym.gcc at mailnull dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67189

            Bug ID: 67189
           Summary: [PATCH] CharsetEncoder changes to support Armed Bear
                    Common Lisp (ABCL)
           Product: classpath
           Version: 0.99
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: classpath
          Assignee: unassigned at gcc dot gnu.org
          Reporter: wispym.gcc at mailnull dot com
  Target Milestone: ---

To get Armed Bear Common Lisp (ABCL) running with GNU classpath, I needed to
patch CharsetEncoder.java as follows. I don't know whether the proper solution
is to change ABCL or to change GNU classpath. I changed GNU classpath.

diff -Naur classpath-0.99/java/nio/charset/CharsetEncoder.java
classpath-0.99.abcl/java/nio/charset/CharsetEncoder.java
--- classpath-0.99/java/nio/charset/CharsetEncoder.java 2010-06-03
20:13:04.000000000 +0100
+++ classpath-0.99.abcl/java/nio/charset/CharsetEncoder.java  2015-08-12
12:16:13.000000000 +0100
@@ -159,8 +159,6 @@
     // in progress" and also that "it resets this Encoder".
     // Should we check to see that the state is reset, or should we
     // call reset()?
-    if (state != STATE_RESET)
-      throw new IllegalStateException ();

     // REVIEW: Using max instead of average may allocate a very large
     // buffer.  Maybe we should do something more efficient?
@@ -199,9 +197,6 @@
     // a return value indicating an incomplete decoding operation"
     // XXX: We will not check the previous return value, just
     // that the previous call passed true for endOfInput
-    if (state != STATE_RESET && state != STATE_CODING
-        && !(endOfInput && state == STATE_END))
-      throw new IllegalStateException ();
     state = newState;

     for (;;)