> On Nov 3, 2016, at 4:53 AM, Roman Grigoriadi <roman.grigori...@oracle.com>
> wrote:
>
> CCC for CommandInfo changes is now approved
> (http://ccc.us.oracle.com/8047773), you can find new webrev here:
>
> http://cr.openjdk.java.net/~aefimov/8164479/03/
>
javax/activation/CommandInfo.java
92 * When java.beans.Beans is not visible (when java.desktop module is not
Formatting nit: “java.beans.Beans” and “java.desktop” can be wrapped with
{@code…}
218 Class<?> beanClass = Class.forName(cn, true, loader);
I suggest to not to initialize the class in the Class::forName call (i.e.
s/true/false)
220 return beanClass.newInstance();
Class::newInstance is deprecated in JDK 9. Can you replace it with
Class::getDeclaredConstructor and Constructor::newInstance
What does wsimport and xjc -m option do?
67 private static final String JAXWS_PACKAGE = "java.xml.ws”;
- Is it a module name, not package name?
Does java.xml.ws module still require java.rmi? Should it be updated?
Otherwise, looks okay to me.
Mandy
