diff options
author | Mathieu Chartier <mathieuc@google.com> | 2013-09-17 15:17:28 -0700 |
---|---|---|
committer | Mathieu Chartier <mathieuc@google.com> | 2013-09-17 16:31:24 -0700 |
commit | 6aa3df965395566ed6a4fec4af37c2b7577992e9 (patch) | |
tree | c17406cf6cbea16165ad05130609450ff0e1e204 /runtime/intern_table.cc | |
parent | cb135ad769a5368f6ceb8caf25105a95580cbd6a (diff) | |
download | art-6aa3df965395566ed6a4fec4af37c2b7577992e9.zip art-6aa3df965395566ed6a4fec4af37c2b7577992e9.tar.gz art-6aa3df965395566ed6a4fec4af37c2b7577992e9.tar.bz2 |
Refactor system weak sweeping, add support for modification.
Required for moving collectors.
Change-Id: Ib97ba4a05af1139f8d388077a15e62bcb9534855
Diffstat (limited to 'runtime/intern_table.cc')
-rw-r--r-- | runtime/intern_table.cc | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/runtime/intern_table.cc b/runtime/intern_table.cc index 6b0a51b..cfed9ab 100644 --- a/runtime/intern_table.cc +++ b/runtime/intern_table.cc @@ -194,14 +194,16 @@ bool InternTable::ContainsWeak(mirror::String* s) { return found == s; } -void InternTable::SweepInternTableWeaks(IsMarkedTester is_marked, void* arg) { +void InternTable::SweepInternTableWeaks(RootVisitor visitor, void* arg) { MutexLock mu(Thread::Current(), intern_table_lock_); - // TODO: std::remove_if + lambda. for (auto it = weak_interns_.begin(), end = weak_interns_.end(); it != end;) { mirror::Object* object = it->second; - if (!is_marked(object, arg)) { + mirror::Object* new_object = visitor(object, arg); + if (new_object == nullptr) { + // TODO: use it = weak_interns_.erase(it) when we get a c++11 stl. weak_interns_.erase(it++); } else { + it->second = down_cast<mirror::String*>(new_object); ++it; } } |