aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/test/java
diff options
context:
space:
mode:
authorGravatar Ulf Adams <ulfjack@google.com>2016-08-11 12:39:46 +0000
committerGravatar Yue Gan <yueg@google.com>2016-08-12 08:49:35 +0000
commit653a16a49b83bd96b18ee71c8123f53305811472 (patch)
tree64a878f636aff27a4448caa5827a4b0852336093 /src/test/java
parent67ad82a319ff8959e69e774e7c15d3af904ec23d (diff)
Replace calls to <class>.newInstance with <class>.getConstructor().newInstance.
This is safer; newInstance on class objects bypasses exception checking. -- MOS_MIGRATED_REVID=129976805
Diffstat (limited to 'src/test/java')
-rw-r--r--src/test/java/com/google/devtools/build/lib/buildtool/util/TestRuleModule.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/util/TestRuleModule.java b/src/test/java/com/google/devtools/build/lib/buildtool/util/TestRuleModule.java
index 58de3682d9..4f2a66eb3f 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/util/TestRuleModule.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/util/TestRuleModule.java
@@ -24,7 +24,7 @@ public class TestRuleModule {
public static BlazeModule getModule() {
try {
Class<?> moduleClass = Class.forName(TestConstants.TEST_RULE_MODULE);
- return (BlazeModule) moduleClass.newInstance();
+ return (BlazeModule) moduleClass.getConstructor().newInstance();
} catch (Exception e) {
throw new IllegalStateException(e);
}