slawekjaranowski commented on code in PR #84:
URL: https://github.com/apache/maven-clean-plugin/pull/84#discussion_r1938483270
##########
src/main/java/org/apache/maven/plugins/clean/Cleaner.java:
##########
@@ -309,24 +315,23 @@ private int delete(Path file, boolean failOnError,
boolean retryOnError) throws
for (int delay : delays) {
try {
Thread.sleep(delay);
- } catch (InterruptedException e2) {
- exception.addSuppressed(e2);
+ } catch (InterruptedException e) {
+ Thread.currentThread().interrupt();
Review Comment:
As we don't re-throw InterruptedException here we should set interrupted
flag agin
http://www.javapractices.com/topic/TopicAction.do?Id=251
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
