[JAXP] RFR: 8169778: Add new public methods to get new instances of the JAXP factories builtin system-default implementations

[Joe Wang]

On 11/16/16, 8:03 AM, Roger Riggs wrote:
> Hi Daniel,
>
> DocumentBuilderFactory and classes:
>    - The new methods uses "Creates" instead of "Obtain";
>      perhaps it should be consistent with existing newInstance methods?

"Obtain" was used in the early version of JAXP, and "Creates" in newer 
StAX factories. I used "Creates" as an example to explain the norm of 
writing Javadoc (the description is in 3rd person). If we want the 
consistency, we could do "Obtains" in the JAXP factories and "Creates" 
in StAX, alternatively, we could change all of them.

Regards,
Joe

>
>    -  "Otherwise," and "Otherwise" (with and without ",") are not 
> consistent in the webrev.
>
>
> XMLInputFactory.java:
>   -line 212: the "system-default" should be inside the braces with the 
> @link.
>
> SchemaFactory.java:
>   - line 201:  (pre-existing typo) "in a implementation" -> "in an 
> implementation"
>
> XPathFactory.java
>  - line 177: The {@linkplain platform...} doesn't look like a properly 
> formed link.
>
>
> XMLOutputFactoryNewInstanceTest
>  - line 2, has a 1999 date but is a new file.
>
> Many of the files have long lines (some new) that make side-by-side 
> compares not fit on the screen.
>
> Regards, Roger
>
>
>
>
> On 11/16/2016 10:24 AM, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find below a patch for:
>> 8169778: Add new public methods to get new instances of the
>>          JAXP factories builtin system-default implementations
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8169778
>> webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8169778/webrev.00/
>>
>> best regards,
>>
>> -- daniel
>