[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wink-dev
Subject:    [jira] Resolved: (WINK-202) Change XmlFormattingOptions to not do
From:       "Bryant Luk (JIRA)" <jira () apache ! org>
Date:       2009-09-26 20:58:16
Message-ID: 427108493.1253998696011.JavaMail.jira () brutus
[Download RAW message or body]


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


[prev in list] [next in list] [prev in thread] [next in thread]