-
Notifications
You must be signed in to change notification settings - Fork 28
Open
Labels
bugSomething isn't workingSomething isn't workingpriority:majorMajor loss of functionMajor loss of function
Description
Elliotte Rusty Harold opened MSHARED-980 and commented
Properly handled, there is never any need to specify the character encoding when reading an XML document. XML documents fully identify their character encoding. The developer at this level doesn't need to know and shouldn't think about the character encoding.
That is, these methods are wrong:
public static Xpp3Dom build( @WillClose InputStream is, @Nonnull String encoding )
public static Xpp3Dom build( @WillClose InputStream is, @Nonnull String encoding, boolean trim )
There should be no encoding argument. Instead the XML should be read from an InputStream without converting it to a reader first. Let the XML parser detect the encoding.
No further details from MSHARED-980
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingpriority:majorMajor loss of functionMajor loss of function