Thank you for review.

Leonid

> On Oct 2, 2019, at 11:14 AM, Hohensee, Paul <[email protected]> wrote:
> 
> Looks good.
>  
> Paul
>  
> From: serviceability-dev <[email protected] 
> <mailto:[email protected]>> on behalf of Leonid 
> Mesnik <[email protected] <mailto:[email protected]>>
> Date: Wednesday, October 2, 2019 at 11:12 AM
> To: serviceability-dev <[email protected] 
> <mailto:[email protected]>>
> Subject: [UNVERIFIED SENDER] RFR(XXS): 8231768: Remove duplicate if/else in 
> vmTestbase/nsk/share/jdi/Binder.java
>  
> Hi 
>  
> Could you please review following tiny and trivial fix which just remove 
> duplicated if/else branch in nsk/share/jdi/Binder class.
>  
> Verified that all tests still pass locally.
>  
> webrev: http://cr.openjdk.java.net/~lmesnik/8231768/webrev.00/ 
> <http://cr.openjdk.java.net/~lmesnik/8231768/webrev.00/>
> bug: https://bugs.openjdk.java.net/browse/JDK-8231768 
> <https://bugs.openjdk.java.net/browse/JDK-8231768>
>  
> Leonid

Reply via email to