On Thu, 24 Mar 2022 06:45:46 GMT, Xue-Lei Andrew Fan <xue...@openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/ssl/SSLEngineImpl.java line 802:
>> 
>>> 800:             } finally {
>>> 801:                 engineLock.unlock();
>>> 802:             }
>> 
>> I looked the update further.  It would be nice if the try-statement could 
>> support more than one finally blocks in the future so that the code could be 
>> more clear.
>> 
>> try {
>>     ...
>> } finally {
>>    // the 1st final block
>> } finally {
>>    // the 2nd final block
>> }
>> 
>> 
>> For the block from 781 to 787, it may be more effective if moving the block 
>> out of the synchronization lock (that's, moving 781-787 to line 779.)
>
>> @XueleiFan, I'm not following your first comment.
> 
> Sorry for that.  I was wondering something new in the future, which is not 
> doable in the current Java language.  Please just leave it alone.

Ah!  Thanks.

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

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

Reply via email to