On Fri, 6 Jan 2023 19:56:20 GMT, Justin Lu <d...@openjdk.org> wrote:
>> Removed constructors of primitive wrapper classes (deprecated for removal)
>> in _javax.xml.stream.XMLOutputFactory_
>>
>> Replaced with Boolean static fields: Boolean.TRUE and Boolean.FALSE
>
> src/java.xml/share/classes/javax/xml/stream/XMLOutputFactory.java line 59:
>
>> 57: *
>> 58: * <p>The property can be set with the following code line:
>> 59: * {@code setProperty("javax.xml.stream.isRepairingNamespaces",
>> Boolean.TRUE|Boolean.FALSE);}
>
> From a code standpoint I don't see why the OR operation is needed (Also,
> shouldn't a Conditional OR be used over a Bitwise OR). Is it to emphasize
> that true _or_ false can be passed as a parameter?
I think the later, that the value can be true or false. As an example, one
value is fine, e.g.
{@code setProperty("javax.xml.stream.isRepairingNamespaces", Boolean.TRUE);}
or
{@code setProperty("javax.xml.stream.isRepairingNamespaces", true);}
let autoboxing handle it.
-------------
PR: https://git.openjdk.org/jdk/pull/11872
