Public Comment

[comment-form@oasis-open.org]

Comment from: elharo@metalab.unc.edu

Many tests indent output unnecessarily. i.e. when they are not specifically testing indent="yes". This makes it difficult to compare results between different processors because they're allowed to indent differently or not indent at all, and between serializing and non-serializing processors. Specifically this is an issue with these Xalan conformance tests:

attribset/attribset41.xsl
attribset/attribset42.xsl
attribset/attribset43.xsl
attribset/attribset44.xsl
axes/axes01.xsl
axes/axes129.xsl
axes/axes130.xsl
copy/copy09.xsl
copy/copy49.xsl
idkey/idkey25.xsl
idkey/idkey59.xsl
lre/lre15.xsl
lre/lre17.xsl
lre/lre18.xsl
namespace/namespace35.xsl
namespace/namespace140.xsl
predicate/predicate38.xsl
reluri/reluri09.xsl
reluri/reluri10.xsl
reluri/reluri11.xsl
string/string129.xsl
variable/variable11.xsl
variable/variable22.xsl
variable/variable27.xsl
variable/variable28.xsl
variable/variable29.xsl
variable/variable30.xsl
variable/variable56.xsl

I suggest these be rewritten without <xsl:output indent="yes"/> 

There are also a couple of cases in the whitespace directory but these are intended to test indenting.