RFR: 8273553: sun.security.ssl.SSLEngineImpl.closeInbound also has similar error of JDK-8253368 [v2]

[Bradford Wetmore]

On Thu, 24 Mar 2022 06:45:46 GMT, Xue-Lei Andrew Fan <xuelei at 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