summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2022-04-01 08:18:31 +0200
committerGitHub <noreply@github.com>2022-04-01 08:18:31 +0200
commitd72046e0f325ab96f30c45aa15c8d0f95ebe1ed2 (patch)
tree44de72ae5c5895411db4bd74e7db045dd4799510
parente21ecbd7cfda3ab8de9219abd4bf42723f7c382b (diff)
parentdf2fa80f10ee2f9516330a58de2abe2f206b5387 (diff)
Merge pull request #21929 from vespa-engine/suppress-removal-for-SecurityManager
-rw-r--r--searchlib/src/test/java/com/yahoo/searchlib/gbdt/GbdtConverterTestCase.java3
-rw-r--r--vespajlib/src/main/java/com/yahoo/concurrent/ThreadFactoryFactory.java1
-rw-r--r--vespajlib/src/test/java/com/yahoo/io/FatalErrorHandlerTestCase.java5
3 files changed, 8 insertions, 1 deletions
diff --git a/searchlib/src/test/java/com/yahoo/searchlib/gbdt/GbdtConverterTestCase.java b/searchlib/src/test/java/com/yahoo/searchlib/gbdt/GbdtConverterTestCase.java
index fa54af25481..d846d322720 100644
--- a/searchlib/src/test/java/com/yahoo/searchlib/gbdt/GbdtConverterTestCase.java
+++ b/searchlib/src/test/java/com/yahoo/searchlib/gbdt/GbdtConverterTestCase.java
@@ -22,11 +22,13 @@ import static org.junit.Assert.fail;
public class GbdtConverterTestCase {
@Before
+ @SuppressWarnings("removal")
public void enableSecurityManager() {
System.setSecurityManager(new NoExitSecurityManager());
}
@After
+ @SuppressWarnings("removal")
public void disableSecurityManager() {
System.setSecurityManager(null);
}
@@ -140,6 +142,7 @@ public class GbdtConverterTestCase {
}
}
+ @SuppressWarnings("removal")
private static class NoExitSecurityManager extends SecurityManager {
@Override
diff --git a/vespajlib/src/main/java/com/yahoo/concurrent/ThreadFactoryFactory.java b/vespajlib/src/main/java/com/yahoo/concurrent/ThreadFactoryFactory.java
index 1779179efcd..a2478046a9e 100644
--- a/vespajlib/src/main/java/com/yahoo/concurrent/ThreadFactoryFactory.java
+++ b/vespajlib/src/main/java/com/yahoo/concurrent/ThreadFactoryFactory.java
@@ -36,6 +36,7 @@ public class ThreadFactoryFactory {
final String namePrefix;
final boolean isDaemon;
+ @SuppressWarnings("removal")
Factory(final String name, boolean isDaemon) {
this.isDaemon = isDaemon;
final SecurityManager s = System.getSecurityManager();
diff --git a/vespajlib/src/test/java/com/yahoo/io/FatalErrorHandlerTestCase.java b/vespajlib/src/test/java/com/yahoo/io/FatalErrorHandlerTestCase.java
index 1999323af6b..dab91b6a995 100644
--- a/vespajlib/src/test/java/com/yahoo/io/FatalErrorHandlerTestCase.java
+++ b/vespajlib/src/test/java/com/yahoo/io/FatalErrorHandlerTestCase.java
@@ -12,9 +12,10 @@ import org.junit.Test;
/**
* Just to remove noise from the coverage report.
*
- * @author <a href="mailto:steinar@yahoo-inc.com">Steinar Knutsen</a>
+ * @author Steinar Knutsen
*/
public class FatalErrorHandlerTestCase {
+ @SuppressWarnings("removal")
private static final class AvoidExiting extends SecurityManager {
@Override
@@ -31,12 +32,14 @@ public class FatalErrorHandlerTestCase {
private FatalErrorHandler h;
@Before
+ @SuppressWarnings("removal")
public void setUp() throws Exception {
h = new FatalErrorHandler();
System.setSecurityManager(new AvoidExiting());
}
@After
+ @SuppressWarnings("removal")
public void tearDown() throws Exception {
System.setSecurityManager(null);
}