Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (athena.apache.org: domain of smallsmallorgan@gmail.com
 designates 64.233.166.183 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references;
        b=CgBBvz9f3Xw0pG+R4qi85DZ3y88RGFArWnik4cZNMvNjTcAAssEj6D0v+A1rYaeqqPC1rY9e0y4Et1CAFWTKihu9/2rNcXBfd5lZeTSbAJnb9U0LYx5fp1kVrlBWShfF7oFE4oBDigkePgcasIlifERaoDUHQ2VrnH/REYDsL/4=
Message-ID: <473c46620708151849j336a4782t45475d6c5c9cd3ef@mail.gmail.com>
Date: Thu, 16 Aug 2007 09:49:14 +0800
From: "Spark Shen" <smallsmallorgan@gmail.com>
To: dev@harmony.apache.org
Subject: Re: [classlib][beans] Another non-bug difference? (was [JIRA 1568]
 [classlib][beans] XMLDecoder(3 params).getExceptionListener() returns object
 on RI and 'null' on Harmony)
In-Reply-To: <f982cec50708150113p45dd2b6emf30500f5124c35db@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_54824_12812141.1187228954456"
References: <473c46620708150016l1b648e13m511e4071915eb82f@mail.gmail.com>
	 <f982cec50708150058s70d04e63qad247a4444665246@mail.gmail.com>
	 <473c46620708150101s61d8b3aclf287a427e85ae02d@mail.gmail.com>
	 <f982cec50708150113p45dd2b6emf30500f5124c35db@mail.gmail.com>

------=_Part_54824_12812141.1187228954456
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

If delaying inputStream parsing to readObject method. Then a boolean
variable is needed to guarantee that parsing is done only once.  The
potential drawback is that, original implementation is instinctive thread
safe regarding to inputStream parsing, while using boolean we may need extra
efforts to guarantee thread safety or just give it up.2007/8/15, Yang Paulex
<paulex.yang@gmail.com>:
>
> 2007/8/15, Spark Shen <smallsmallorgan@gmail.com>:
> >
> > RI will print out 'res = class java.beans.Statement$1', with on
> exception
> > information.
> >
> > 2007/8/15, Yang Paulex <paulex.yang@gmail.com>:
> > >
> > > 2007/8/15, Spark Shen <smallsmallorgan@gmail.com>:
> > > >
> > > > Hi, all
> > > > Seems JIRA 1568 needs further attention.
> > > >
> > > > Even fixed 1568, harmony still prints some shocking stack trace. The
> > > test
> > > > case[1] given by 1568 will reveal this. Although this can be viewed
> as
> > a
> > > > non-bug difference, print such shocking error message when
> > constructing
> > > a
> > > > XMLDecoder may not be so elegant.
> > > > I have two alternatives to amend this behavior:
> > > >
> > > > 1. simplify the error message printed out by
> DefaultExceptionListener,
> > > as
> > > > the patch given below:
> > > > Index: beans/src/main/java/java/beans/XMLDecoder.java
> > > > ===================================================================
> > > > --- beans/src/main/java/java/beans/XMLDecoder.java    (revision
> > 566039)
> > > > +++ beans/src/main/java/java/beans/XMLDecoder.java    (working copy)
> > > > @@ -45,9 +45,10 @@
> > > >
> > > >      private static class DefaultExceptionListener implements
> > > > ExceptionListener {
> > > >
> > > > +        @SuppressWarnings("nls")
> > > >          public void exceptionThrown(Exception e) {
> > > > -            e.printStackTrace();
> > > > -            System.err.println("Continue..."); //$NON-NLS-1$
> > > > +            System.err.println(e.getClass().getCanonicalName() + ":
> "
> > > > +e.getMessage());
> > > > +            System.err.println("Continue...");
> > > >          }
> > > >      }
> > > >
> > > >
> > > >
> > > > 2. Delay inputStream parsing, moving it from constructor to other
> > place
> > > > like
> > > > readObject.
> > > >
> > > > Or any other suggestions?
> > >
> > >
> > > What's RI's behavior here?
> > >
> > > [1] test case given by
> > https://issues.apache.org/jira/browse/HARMONY-1568
> > > > import java.beans.*;
> > > > import java.io.ByteArrayInputStream;
>
>
> I think we can just follow RI here. Any drawbacks?
>
> > >
> > > > public class test {
> > > >    public static void main(String[] args) {
> > > >        System.out.println("res = " + new XMLDecoder(new
> > > > ByteArrayInputStream(new byte[] {1, 2}), new Double(1),
> > > > null).getExceptionListener());
> > > >     }
> > > > }
> > > >
> > > > --
> > > > Spark Shen
> > > > China Software Development Lab, IBM
> > > >
> > >
> > >
> > >
> > > --
> > > Paulex Yang
> > > China Software Development laboratory
> > > IBM
> > >
> >
> >
> >
> > --
> > Spark Shen
> > China Software Development Lab, IBM
> >
>
>
>
> --
> Paulex Yang
> China Software Development laboratory
> IBM
>


-- 
Spark Shen
China Software Development Lab, IBM

------=_Part_54824_12812141.1187228954456--