tree be03b4e19f663cc644b866eeb3a3cca3f63b1483
parent 1887559eb510abb420ccd96823f21f16eb77c691
author Max Kellermann <mk@cm4all.com> 1614520737 +0100
committer GitHub <noreply@github.com> 1614520737 -0500
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgO6GhCRBK7hj4Ov3rIwAAdHIIAAhkhfNHKtpDXhlrh4M58lNu
 2vs6S5IWeDcdksOsX9mv8SRxCOX+QmrA59IRZZra4teulWzsdoPm6PYkf1iZq81l
 6T1+13a9Nu1AWplGLjJpwqcYNMyE7XCqtgBEJSL7wqqznDf+4VxeHmgpsFZpI7U4
 3wDb7SPeK1O6Jr7xPwxZqT8t0Yw+8WopikECTu9FqOBJvVj2NVdUxJafSuiVCQq7
 YbAi++FNvZRyN1oHwBfVuHw+20qGAC3/WlaavN+3h9vWztorhIHLS7mDpTHRFA49
 fUWGsaFDLsjZOZ+a1y9Ml8fuxG7MwufN12Wwc2cSRMnOSYZZ9Oimn8lCK+ckkyo=
 =/gWf
 -----END PGP SIGNATURE-----
 

Rework SoftRefFilesCache locking part 1 (#158)

* SoftRefFilesCache: don't call super.close()

AbstractVfsComponent.close() is documented to do nothing, so there's
no point in calling it.

* SoftRefFilesCache: use isEmpty() instead of "size()<1"

For some containers, size() is O(n), but isEmpty() is always O(1).

* SoftRefFilesCache: remove redundant isInterrupted() check

The following ReferenceQueue.remove() call will do that check again.

* SoftRefFilesCache: simplify the InterruptedException catch block

By reversing the order of "while" and "try/catch", we can simply omit
the "break" pseudo-goto statement.