Hi Vipin,
I've only looked at the management files. They look good in general.
Could you, please, fix the indentation?
Thanks,
Serguei
On 9/9/20 12:38, Vipin Sharma wrote:
I've only looked at the management files. They look good in general.
src/java.management/share/classes/java/lang/management/ClassLoadingMXBean.java108 * @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.java286 * @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