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

[Lance Andersen]

> On Nov 16, 2016, at 12:18 PM, Joe Wang <huizhe.wang at oracle.com> wrote:
> 
> 
> 
> 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.

I would work towards consistency throughout.  Personally I prefer “Creates” over “Obtains”


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

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>