On Sat, 23 Oct 2021 12:51:15 GMT, Andrey Turbanov <d...@openjdk.java.net> wrote:

> 8276042: Remove unused local variables in java.naming

Hi Andrey,

Thanks for cleaning up the code.
Changes look good to me, with one suggestion below.

src/java.naming/share/classes/com/sun/jndi/toolkit/ctx/PartialCompositeContext.java
 line 514:

> 512:         Object obj = cont.getResolvedObj();
> 513: 
> 514:         if (obj instanceof PartialCompositeContext) {

Since we're changing this method, maybe `instanceof` pattern matching can be 
used here to remove casting below.

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

Marked as reviewed by aefimov (Committer).

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

Reply via email to