[
https://issues.apache.org/jira/browse/WINK-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12757394#action_12757394
]
Bryant Luk commented on WINK-202:
---------------------------------
I added a XmlDefaultOptionsContextResolver loaded by default for a few of the
existing JAXB classes that had tests that explicitly wanted the fragment. For
performance purposes AtomFeed/AtomEntry should probably be removed eventually
but wanted to make sure no one saw any issues first.
> Change XmlFormattingOptions to not do anything by default
> ---------------------------------------------------------
>
> Key: WINK-202
> URL: https://issues.apache.org/jira/browse/WINK-202
> Project: Wink
> Issue Type: Improvement
> Components: Common
> Affects Versions: 0.2
> Reporter: Bryant Luk
> Assignee: Bryant Luk
> Fix For: 0.2
>
>
> Changing XmlFormattingOptions to not do anything by default to help improve
> minor compatibility and gain a "cheap" performance benefit (up to 2%
> overall). While it's a nice feature to have, it may be unexpected for some
> clients to leave out the XML declaration. Furthermore, while the indenting
> is beneficial for debugging, generally people will not be reading the XML.
> The change should be relatively simple I think:
> {code}
> if (formatingOptions != null) {
> JAXBUtils.setXmlFormattingOptions(marshaller, formatingOptions);
> } else {
> JAXBUtils.setXmlFormattingOptions(marshaller, XmlFormattingOptions
> .getDefaultXmlFormattingOptions());
> }
> {code}
> should get rid of the else statement. This would allow the marshaller to do
> whatever the "natural" JDK defaults are automatically.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
