elharo commented on code in PR #135:
URL: https://github.com/apache/xerces-j/pull/135#discussion_r4516912516
##########
src/org/apache/xml/serialize/BaseMarkupSerializer.java:
##########
@@ -593,14 +593,13 @@ public void ignorableWhitespace( char[] chars, int start,
int length )
try {
content();
- // Print ignorable whitespaces only when indenting, after
- // all they are indentation. Cancel the indentation to
- // not indent twice.
+ // When indenting, ignorable whitespace is indentation. Cancel
+ // the indentation to not indent twice.
Review Comment:
or did you mean the indentation? In that case, double indenting is simply
not what the client asked for.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
