RFR: 8261938: ASN1Formatter.annotate should not return in the finally block [v2]

Roger Riggs rriggs at openjdk.java.net
Thu Feb 18 14:55:47 UTC 2021


On Thu, 18 Feb 2021 02:46:53 GMT, Jie Fu <jiefu at openjdk.org> wrote:

>> Hi all,
>> 
>> ASN1Formatter.annotate should be able to throw an IOException according to this comment [1].
>> But it fails to do so due to the return [2] in the finally block, which would swallow the IOException.
>> 
>> Generally, it seems not good to put a return statement in a finally block because it would overwrite other return-statements or Exceptions [3].
>> 
>> Thanks.
>> Best regards,
>> Jie
>> 
>> [1] https://github.com/openjdk/jdk/blob/master/test/lib/jdk/test/lib/hexdump/ASN1Formatter.java#L113
>> [2] https://github.com/openjdk/jdk/blob/master/test/lib/jdk/test/lib/hexdump/ASN1Formatter.java#L120
>> [3] https://stackoverflow.com/questions/17481251/finally-block-does-not-complete-normally-eclipse-warning
>
> Jie Fu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update the copyright year

An EOFException can occur during the call to annotate() and must return the accumulated contents of the StringBuffer.  Otherwise it is discarded.

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

Changes requested by rriggs (Reviewer).

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


More information about the core-libs-dev mailing list