You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Gary D. Gregory (Jira)" <ji...@apache.org> on 2021/09/26 14:24:00 UTC
[jira] [Commented] (IO-729) BrokenReader, BrokenWriter,
BrokenInputStream, BrokenOutputStream can cause IllegalArgumentException in
combination with try-with-resources
[ https://issues.apache.org/jira/browse/IO-729?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17420303#comment-17420303 ]
Gary D. Gregory commented on IO-729:
------------------------------------
Hi [~Spoor],
I am not reading a proposal here. May you please propose a PR on GitHub?
> BrokenReader, BrokenWriter, BrokenInputStream, BrokenOutputStream can cause IllegalArgumentException in combination with try-with-resources
> -------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: IO-729
> URL: https://issues.apache.org/jira/browse/IO-729
> Project: Commons IO
> Issue Type: Bug
> Components: Streams/Writers
> Affects Versions: 2.8.0
> Reporter: Rob Spoor
> Priority: Minor
>
> The following little code snippet can cause an IllegalArgumentException with message "Self-suppression not permitted":
> {code} try (Writer writer = new BrokenWriter()) {
> writer.write('a');
> }{code}
> The try-with-resources mechanism will try to add the exception thrown from {{close}} as suppress exception to the exception thrown from {{write}}. Since those are the same, an exception is thrown. From the source of java.lang.Throwable:
> {code} public final synchronized void addSuppressed(Throwable exception) {
> if (exception == this)
> throw new IllegalArgumentException(SELF_SUPPRESSION_MESSAGE, exception);{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)