Closed
Bug 128468
Opened 23 years ago
Closed 22 years ago
java.io.NotSerializableException: org.mozilla.javascript.NativeError
Categories
(Rhino Graveyard :: Core, defect)
Tracking
(Not tracked)
VERIFIED
INVALID
1.5R4
People
(Reporter: andrew, Assigned: norrisboyd)
Details
While org.mozilla.javascript.EcmaError is a subclass of
java.lang.RuntimeException and thus Serializable, it contains a
org.mozilla.javascript.NativeError that is not serializable. As a result,
receive a NotSerializableException when serializing an EcmaError
Comment 1•22 years ago
|
||
I agree this is annoying especially when rhino is executed remotly (RMI/J2EE). I
don't see any problems with making NativeError serializeable, but maybe there is?
[mmoeller@mmoeller rhino]$ cvs diff -u src/org/mozilla/javascript/NativeError.java
Index: src/org/mozilla/javascript/NativeError.java
===================================================================
RCS file: /cvsroot/mozilla/js/rhino/src/org/mozilla/javascript/NativeError.java,v
retrieving revision 1.23
diff -u -r1.23 NativeError.java
--- src/org/mozilla/javascript/NativeError.java 25 Nov 2002 16:44:30 -0000 1.23
+++ src/org/mozilla/javascript/NativeError.java 25 Nov 2002 18:40:09 -0000
@@ -43,7 +43,7 @@
*
* ECMA 15.11
*/
-final class NativeError extends IdScriptable {
+final class NativeError extends IdScriptable implements java.io.Serializable {
static void init(Context cx, Scriptable scope, boolean sealed) {
NativeError obj = new NativeError();
Comment 2•22 years ago
|
||
cc'ing Igor -
Comment 3•22 years ago
|
||
NativeError inherits from IdScriptable which is a subclass of ScriptableObject
whih implements Serializable interface since 1.5R3 release.
I suggest to mark this as invalid unless detailed test/Rhino version case is
provided.
Comment 4•22 years ago
|
||
Marking INVALID since the 1.5R3 release should be OK on this.
Note: this is obtainable at ftp://ftp.mozilla.org/pub/js, along
with an even more recent version of Rhino.
Andrew: please reopen if this is still a problem with the
new releases; thanks -
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•