Hi Vipin,

I've only looked at the management files. They look good in general.

src/java.management/share/classes/java/lang/management/ClassLoadingMXBean.java
108      * @throws  java.lang.SecurityException if a security manager
109      *             exists and the caller does not have
110      *             ManagementPermission("control").

src/java.management/share/classes/java/lang/management/MemoryMXBean.java
286      * @throws  java.lang.SecurityException if a security manager
287      *             exists and the caller does not have
288      *             ManagementPermission("control").

Could you, please, fix the indentation?

Thanks,
Serguei


On 9/9/20 12:38, Vipin Sharma wrote:
Updated @exception with @throws for core-libs, it fixes all open sub-tasks of JDK-8252536.

Open Subtasks part of this fix are:
1. JDK-8252537
2. JDK-8252539
3. JDK-8252540
4. JDK-8252541

-------------

Commit messages:
 - JDK-8252537 Updated @exception with @throws

Changes: https://git.openjdk.java.net/jdk/pull/95/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=95&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8252537
  Stats: 1585 lines in 86 files changed: 0 ins; 0 del; 1585 mod
  Patch: https://git.openjdk.java.net/jdk/pull/95.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/95/head:pull/95

PR: https://git.openjdk.java.net/jdk/pull/95

Reply via email to