Re: RFR: 8276042: Remove unused local variables in java.naming [v2]

2021-10-27 Thread Vyom Tewari
On Wed, 27 Oct 2021 15:42:32 GMT, Andrey Turbanov wrote: >> 8276042: Remove unused local variables in java.naming > > Andrey Turbanov has updated the pull request incrementally with one > additional commit since the last revision: > > 8276042: Remove unused local variables in java.naming >

Re: RFR: 8276042: Remove unused local variables in java.naming [v2]

2021-10-27 Thread Daniel Fuchs
On Wed, 27 Oct 2021 15:42:32 GMT, Andrey Turbanov wrote: >> 8276042: Remove unused local variables in java.naming > > Andrey Turbanov has updated the pull request incrementally with one > additional commit since the last revision: > > 8276042: Remove unused local variables in java.naming >

Re: RFR: 8276042: Remove unused local variables in java.naming [v2]

2021-10-27 Thread Aleksei Efimov
On Wed, 27 Oct 2021 15:42:32 GMT, Andrey Turbanov wrote: >> 8276042: Remove unused local variables in java.naming > > Andrey Turbanov has updated the pull request incrementally with one > additional commit since the last revision: > > 8276042: Remove unused local variables in java.naming >

Re: RFR: 8276042: Remove unused local variables in java.naming [v2]

2021-10-27 Thread Andrey Turbanov
On Wed, 27 Oct 2021 11:08:31 GMT, Aleksei Efimov wrote: >> Andrey Turbanov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8276042: Remove unused local variables in java.naming >> use instanceof pattern > >

Re: RFR: 8276042: Remove unused local variables in java.naming [v2]

2021-10-27 Thread Andrey Turbanov
> 8276042: Remove unused local variables in java.naming Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision: 8276042: Remove unused local variables in java.naming use instanceof pattern - Changes: - all:

Re: RFR: 8276042: Remove unused local variables in java.naming

2021-10-27 Thread Aleksei Efimov
On Sat, 23 Oct 2021 12:51:15 GMT, Andrey Turbanov 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

RFR: 8276042: Remove unused local variables in java.naming

2021-10-27 Thread Andrey Turbanov
8276042: Remove unused local variables in java.naming - Commit messages: - [PATCH] Remove unused local variable in java.naming Changes: https://git.openjdk.java.net/jdk/pull/6091/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6091=00 Issue: