<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">Hi Vipin,<br>
<br>
I've only looked at the management files. They look good in
general.<br>
<br>
<code></code><span class="file-modified">src/java.management/share/classes/java/lang/management/ClassLoadingMXBean.java</span><br>
<pre><span id="rhs-hunk-2" class="line-modified">108 * @throws java.lang.SecurityException if a security manager
</span>109 * exists and the caller does not have
110 * ManagementPermission("control").</pre>
<br>
<code></code><span class="file-modified">src/java.management/share/classes/java/lang/management/MemoryMXBean.java</span><br>
<pre><span id="rhs-hunk-2" class="line-modified">286 * @throws java.lang.SecurityException if a security manager
</span>287 * exists and the caller does not have
288 * ManagementPermission("control").</pre>
<span class="file-modified"></span><br>
<span class="file-modified">Could you, please, fix the
indentation?<br>
<br>
Thanks,<br>
Serguei<br>
</span><br>
<br>
On 9/9/20 12:38, Vipin Sharma wrote:<br>
</div>
<blockquote type="cite"
cite="mid:jWf2YS8e4ZhTRfuFPquC4jxUS-urxG4ZDQ2B5FM86rQ=.a0467b19-1e45-4fad-abd6-256f720ef5b7@github.com">
<pre class="moz-quote-pre" wrap="">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: <a class="moz-txt-link-freetext" href="https://git.openjdk.java.net/jdk/pull/95/files">https://git.openjdk.java.net/jdk/pull/95/files</a>
Webrev: <a class="moz-txt-link-freetext" href="https://webrevs.openjdk.java.net/?repo=jdk&pr=95&range=00">https://webrevs.openjdk.java.net/?repo=jdk&pr=95&range=00</a>
Issue: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8252537">https://bugs.openjdk.java.net/browse/JDK-8252537</a>
Stats: 1585 lines in 86 files changed: 0 ins; 0 del; 1585 mod
Patch: <a class="moz-txt-link-freetext" href="https://git.openjdk.java.net/jdk/pull/95.diff">https://git.openjdk.java.net/jdk/pull/95.diff</a>
Fetch: git fetch <a class="moz-txt-link-freetext" href="https://git.openjdk.java.net/jdk">https://git.openjdk.java.net/jdk</a> pull/95/head:pull/95
PR: <a class="moz-txt-link-freetext" href="https://git.openjdk.java.net/jdk/pull/95">https://git.openjdk.java.net/jdk/pull/95</a>
</pre>
</blockquote>
<br>
</body>
</html>