diff options
| author | 2019-05-28 16:39:29 +0100 | |
|---|---|---|
| committer | 2019-05-31 14:15:59 +0000 | |
| commit | 3068d582eff4552ff260d7966fcbdc93e17d0207 (patch) | |
| tree | bc894a414070a06ea2a231fb98607b57b8c3b0cb /runtime/mirror/array.h | |
| parent | 991cd5cc16267b74e390f640eb441102062babb6 (diff) | |
Clean up creating handles from `this`.
Make these member functions static and take an additional
parameter `Handle<.> h_this`. Callers mostly already have
a Handle<> to pass, so we avoid an extra StackHandleScope.
This pattern was already used for some functions.
Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing --interpreter
Change-Id: I4f4478b0526bcb2f3c23305d3b3cc4a65fff9ff5
Diffstat (limited to 'runtime/mirror/array.h')
| -rw-r--r-- | runtime/mirror/array.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/runtime/mirror/array.h b/runtime/mirror/array.h index dbc5d2ac46..19f9a927e3 100644 --- a/runtime/mirror/array.h +++ b/runtime/mirror/array.h @@ -125,8 +125,8 @@ class MANAGED Array : public Object { template<VerifyObjectFlags kVerifyFlags = kDefaultVerifyFlags> ALWAYS_INLINE bool CheckIsValidIndex(int32_t index) REQUIRES_SHARED(Locks::mutator_lock_); - ObjPtr<Array> CopyOf(Thread* self, int32_t new_length) REQUIRES_SHARED(Locks::mutator_lock_) - REQUIRES(!Roles::uninterruptible_); + static ObjPtr<Array> CopyOf(Handle<Array> h_this, Thread* self, int32_t new_length) + REQUIRES_SHARED(Locks::mutator_lock_) REQUIRES(!Roles::uninterruptible_); protected: void ThrowArrayStoreException(ObjPtr<Object> object) REQUIRES_SHARED(Locks::mutator_lock_) |