Re: RFR [13] JDK-8218889: Improperly use of the Optional API

[Bradford Wetmore]

+1.

I noticed this earlier while looking at my other work, and was wondering 
if this was a new style that I had missed.

Thanks,

Brad


On 3/22/2019 9:41 AM, Jamil Nimeh wrote:
This looks fine to me.

--Jamil

On 3/22/19 9:39 AM, Xuelei Fan wrote:
Hi,

Could I have the following update reviewed:
    http://cr.openjdk.java.net/~xuelei/8218889/webrev.00/

In general Optional is a good option.  However, Optional.ofNullable​ 
(null).get() throws NoSuchElementException, as limit the benefits to 
use Optional.ofNullable​() for null object in the session management 
context.   With this update, the Optional is replaced with other 
coding conventions.

Code cleanup, no new regression test.

Thanks,
Xuelei