diff options
author | Andreas Gampe <agampe@google.com> | 2016-11-03 17:24:15 -0700 |
---|---|---|
committer | Andreas Gampe <agampe@google.com> | 2016-11-04 09:11:35 -0700 |
commit | 2e965aca73bacf84123b5c53bb0904b13b48e428 (patch) | |
tree | edad34a00b358ca613b7e18781b91c4473e916fb /compiler/utils/jni_macro_assembler_test.h | |
parent | 2b64cccce7e3697e4ccca44fb95a6cc12c6441aa (diff) |
ART: Use references in assembler tests
Move parameters to const references.
Bug: 32619234
Test: m
Change-Id: Ib68bdc313b91fee1e9e4e1e794eeca630837b005
Diffstat (limited to 'compiler/utils/jni_macro_assembler_test.h')
-rw-r--r-- | compiler/utils/jni_macro_assembler_test.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/compiler/utils/jni_macro_assembler_test.h b/compiler/utils/jni_macro_assembler_test.h index 829f34b4b7..293f4cde9c 100644 --- a/compiler/utils/jni_macro_assembler_test.h +++ b/compiler/utils/jni_macro_assembler_test.h @@ -39,12 +39,12 @@ class JNIMacroAssemblerTest : public testing::Test { typedef std::string (*TestFn)(JNIMacroAssemblerTest* assembler_test, Ass* assembler); - void DriverFn(TestFn f, std::string test_name) { + void DriverFn(TestFn f, const std::string& test_name) { DriverWrapper(f(this, assembler_.get()), test_name); } // This driver assumes the assembler has already been called. - void DriverStr(std::string assembly_string, std::string test_name) { + void DriverStr(const std::string& assembly_string, const std::string& test_name) { DriverWrapper(assembly_string, test_name); } @@ -128,7 +128,7 @@ class JNIMacroAssemblerTest : public testing::Test { virtual void Pad(std::vector<uint8_t>& data ATTRIBUTE_UNUSED) { } - void DriverWrapper(std::string assembly_text, std::string test_name) { + void DriverWrapper(const std::string& assembly_text, const std::string& test_name) { assembler_->FinalizeCode(); size_t cs = assembler_->CodeSize(); std::unique_ptr<std::vector<uint8_t>> data(new std::vector<uint8_t>(cs)); |