ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From apupier <...@git.apache.org>
Subject [GitHub] ant-ivy pull request #62: IVY-1572 - Check for Thread interruption while res...
Date Tue, 06 Feb 2018 15:58:33 GMT
Github user apupier commented on a diff in the pull request:

    https://github.com/apache/ant-ivy/pull/62#discussion_r166349423
  
    --- Diff: src/java/org/apache/ivy/plugins/resolver/AbstractPatternsBasedResolver.java
---
    @@ -91,6 +91,9 @@ protected ResolvedResource findResourceUsingPatterns(ModuleRevisionId
moduleRevi
             Set<String> foundRevisions = new HashSet<>();
             boolean dynamic = getSettings().getVersionMatcher().isDynamic(moduleRevision);
             for (String pattern : patternList) {
    +            if (Thread.currentThread().isInterrupted()) {
    +                return null;
    --- End diff --
    
    the specific exception in this case is in theory https://docs.oracle.com/javase/7/docs/api/java/lang/InterruptedException.html
which is a checked Exception.
    Do you have another runtime exception in mind? Or you want to create a specific Runtime
Exception? I'm wondering what would be the effect in this case to Thread in which it has been
launched. Will it finished also?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message