Sonarqube: Either re-interrupt this method or rethrow the "InterruptedException"

timfel · timfel · commit e830199e69d6 · 2019-04-05T09:33:10.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/SignalModuleBuiltins.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/SignalModuleBuiltins.java
@@ -114,6 +114,7 @@ public void postInitialize(PythonCore core) {
                     poll = signalQueue.poll();
                 }
             } catch (InterruptedException e) {
+                Thread.currentThread().interrupt();
             }
             return poll;
         });
@@ -314,6 +315,7 @@ synchronized static void scheduleAlarm(long seconds) {
                 try {
                     Thread.sleep(seconds * 1000);
                 } catch (InterruptedException e) {
+                    Thread.currentThread().interrupt();
                 }
             }
             sun.misc.Signal.raise(new sun.misc.Signal("ALRM"));

Original file line number	Diff line number	Diff line change
`@@ -114,6 +114,7 @@ public void postInitialize(PythonCore core) {`
`114`	`114`	`poll = signalQueue.poll();`
`115`	`115`	`}`
`116`	`116`	`} catch (InterruptedException e) {`
	`117`	`+ Thread.currentThread().interrupt();`
`117`	`118`	`}`
`118`	`119`	`return poll;`
`119`	`120`	`});`
`@@ -314,6 +315,7 @@ synchronized static void scheduleAlarm(long seconds) {`
`314`	`315`	`try {`
`315`	`316`	`Thread.sleep(seconds * 1000);`
`316`	`317`	`} catch (InterruptedException e) {`
	`318`	`+ Thread.currentThread().interrupt();`
`317`	`319`	`}`
`318`	`320`	`}`
`319`	`321`	`sun.misc.Signal.raise(new sun.misc.Signal("ALRM"));`