On 05/30/2013 06:30 PM, huizhe wang wrote:
On 5/30/2013 9:22 AM, Chris Hegarty wrote:
Looks fine to me Daniel,
I see you have three copies of Objects.java, is it possible to put it
in a common place accessible to all? Or maybe this causes other issues.
JAXP consists of multiple components. The Xerces and Xalan components
specifically are by their own and do not have references across the
boundaries. Our next project is modularization in which Xerces and Xalan
will definitely be split into difference modules. As for BCEL, since
it's always by itself, it's preferable to let it that way, unless it
really affects footprint.
No problem Joe, this addresses my concerns.
-Chris.
-Joe
-Chris.
On 05/30/2013 11:37 AM, Daniel Fuchs wrote:
Hi,
This is a request for review of the backport of
<https://jbs.oracle.com/bugs/browse/JDK-8013900>
to jdk7u-dev.
The changeset for jdk7u-dev [1] had to be altered on one count
compared to the changes pushed in jdk8 [2]:
Since jaxp needs to be compiled using the bootstrap
compiler, it was not possible to use the new
JDK 7 java.util.Objects API.
In order to minimize the changes between the two
changesets - I added an Objects class emulating
the two 'hashCode' and 'equals' methods of java.util.Objects
in the util packages of Xerces, Xalan, and Bcel.
This way - the only thing that needed to change in the rest of
the files was the import clause of the Objects class.
Here is the altered changeset for jdk7u-dev:
[1]
<http://cr.openjdk.java.net/~dfuchs/JDK-8013900/webrev-jdk7u-dev.00/>
Here is the changeset that was pushed in jdk8:
[2] <http://cr.openjdk.java.net/~dfuchs/JDK-8013900/webrev.01/>
-- daniel
Review for the original jdk8 changeset [2] happened on core-libs-dev:
<http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-May/017033.html>