Re: RFR: 8277412: Use String.isBlank to simplify code in sun.net.www.protocol.mailto.Handler

2022-01-26 Thread Daniel Fuchs
On Fri, 12 Nov 2021 19:11:36 GMT, Andrey Turbanov wrote: > All this manually written code actually can be replaced with single > String.isBlank() call. > `file` variable is guaranteed to be non-null. LGTM. Testing didn't reveal any issue. - Marked as reviewed by dfuchs (Reviewer).

Re: RFR: 8277412: Use String.isBlank to simplify code in sun.net.www.protocol.mailto.Handler

2022-01-26 Thread Daniel Fuchs
On Fri, 12 Nov 2021 19:11:36 GMT, Andrey Turbanov wrote: > All this manually written code actually can be replaced with single > String.isBlank() call. > `file` variable is guaranteed to be non-null. I'm running tier1-3 and will approve if the results come green. - PR: https://git

Re: RFR: 8277412: Use String.isBlank to simplify code in sun.net.www.protocol.mailto.Handler

2021-12-07 Thread Andrey Turbanov
On Tue, 7 Dec 2021 08:07:22 GMT, Alan Bateman wrote: >> All this manually written code actually can be replaced with single >> String.isBlank() call. >> `file` variable is guaranteed to be non-null. > > src/java.base/share/classes/sun/net/www/protocol/mailto/Handler.java line 125: > >> 123:

Re: RFR: 8277412: Use String.isBlank to simplify code in sun.net.www.protocol.mailto.Handler

2021-12-07 Thread Alan Bateman
On Fri, 12 Nov 2021 19:11:36 GMT, Andrey Turbanov wrote: > All this manually written code actually can be replaced with single > String.isBlank() call. > `file` variable is guaranteed to be non-null. src/java.base/share/classes/sun/net/www/protocol/mailto/Handler.java line 125: > 123:

RFR: 8277412: Use String.isBlank to simplify code in sun.net.www.protocol.mailto.Handler

2021-11-18 Thread Andrey Turbanov
All this manually written code actually can be replaced with single String.isBlank() call. `file` variable is guaranteed to be non-null. - Commit messages: - [PATCH] Use String.isBlank to simplify code Changes: https://git.openjdk.java.net/jdk/pull/6372/files Webrev: https://webre