Hi Joe,
This looks Ok, a couple of minor comments:
- SerializationHandler.java could use a comment before the method
setIsStandalone().
- ToXMLStream.java seems to have a typo in the comment
/**
331 * Before Xalan 1497, a newline char was printed out if
not inside of an
332 * element. The whitespace is not significant is the
output is standalone
^^^
333 */
Best
lance
On Mar 20, 2012, at 1:32 PM, Joe Wang wrote:
> Hi,
>
> This is an issue introduced by an Apache update that although technically
> correct, was nonetheless an incompatible change. As reported in the CR, it
> caused NetBeans test failures and spurious reformatting of project metadata
> of users' projects that the version control tool would take as if there were
> real changes. The incompatible behavior would more than likely cause many
> problems to users' applications as did to the JAXP SQE tests.
>
> Jesse Glick of the NetBeans tried manually or using 3rd party tool to work
> around the issue, but none satisfactorily.
>
> This patch adds an implementation specific property that can be used to
> essentially neutralize the Apache change to bring back the original behavior.
> It has been tested, to Jesse's satisfactory, to have resolved the NetBeans
> issue with minimal effect in the NetBeans' part.
>
> The use is:
>
> transformer.setOutputProperty("http://www.oracle.com/xml/is-standalone";,
> "yes");
>
> Below is the webrev:
>
> http://cr.openjdk.java.net/~joehw/7u4/cr7150537/webrev/
>
> Please review.
>
> Thanks,
> Joe
>
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com
