diff options
| author | Mathieu Chartier <mathieuc@google.com> | 2014-08-23 16:37:27 -0700 |
|---|---|---|
| committer | Mathieu Chartier <mathieuc@google.com> | 2014-08-23 16:54:24 -0700 |
| commit | bf99f77dda749e2b653e8c45259b1fb56e7bb012 (patch) | |
| tree | 98addbb3533c745781bc26e4a55cbbdd6d6e97e7 /runtime/mirror/class.cc | |
| parent | 762f0a5bdd238458bbb463a9565388f2047a98d4 (diff) | |
| download | art-bf99f77dda749e2b653e8c45259b1fb56e7bb012.zip art-bf99f77dda749e2b653e8c45259b1fb56e7bb012.tar.gz art-bf99f77dda749e2b653e8c45259b1fb56e7bb012.tar.bz2 | |
Clean up Handle usage.
Prefer using ConstHandle instead of Handle as function arguments
since you can't assign new references to ConstHandle which helps
prevent bugs.
Changed NullHandle to be a ConstHandle so that you can never modify
it to be a non null reference.
Change-Id: I81cb979f6f8d5b49e5614966a2caf28c3701dd4f
Diffstat (limited to 'runtime/mirror/class.cc')
| -rw-r--r-- | runtime/mirror/class.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/runtime/mirror/class.cc b/runtime/mirror/class.cc index 5b8eb82..760d54c 100644 --- a/runtime/mirror/class.cc +++ b/runtime/mirror/class.cc @@ -772,7 +772,8 @@ uint16_t Class::GetDirectInterfaceTypeIdx(uint32_t idx) { return GetInterfaceTypeList()->GetTypeItem(idx).type_idx_; } -mirror::Class* Class::GetDirectInterface(Thread* self, Handle<mirror::Class> klass, uint32_t idx) { +mirror::Class* Class::GetDirectInterface(Thread* self, ConstHandle<mirror::Class> klass, + uint32_t idx) { DCHECK(klass.Get() != nullptr); DCHECK(!klass->IsPrimitive()); if (klass->IsArrayClass()) { |