diff options
author | Ulf Adams <ulfjack@google.com> | 2016-08-11 12:39:46 +0000 |
---|---|---|
committer | Yue Gan <yueg@google.com> | 2016-08-12 08:49:35 +0000 |
commit | 653a16a49b83bd96b18ee71c8123f53305811472 (patch) | |
tree | 64a878f636aff27a4448caa5827a4b0852336093 /src/test/java/com/google/devtools/build/lib/buildtool/util | |
parent | 67ad82a319ff8959e69e774e7c15d3af904ec23d (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/com/google/devtools/build/lib/buildtool/util')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/buildtool/util/TestRuleModule.java | 2 |
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); } |