[jira] Resolved: (WINK-202) Change XmlFormattingOptions to not do anything by default

["Bryant Luk (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/WINK-202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bryant Luk resolved WINK-202.
-----------------------------

    Resolution: Fixed

I've fixed the main path for the XmlFormattingOptions and allowed a general properties to be set.  XmlFormattingOptions are still used in a a few of the default public static Marshaller getMarshaller() methods but I don't think that's a problem at this time.

> 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.