archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-09-22 12:04:39 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8025855: Simplify GenRemSet code slightly

Browse source 
Remove a few redundant switch-statements

Reviewed-by: jcoomes, tschatzl
This commit is contained in:
Jesper Wilhelmsson 2013-10-03 13:19:19 +02:00
parent ac15ca0d6a
commit ed022aded1
2 changed files with 4 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

11
hotspot/src/share/vm/memory/collectorPolicy.cpp
View file

@ -137,15 +137,8 @@ bool CollectorPolicy::use_should_clear_all_soft_refs(bool v) {
GenRemSet* CollectorPolicy::create_rem_set(MemRegion whole_heap, GenRemSet* CollectorPolicy::create_rem_set(MemRegion whole_heap,
int max_covered_regions) { int max_covered_regions) {
switch (rem_set_name()) { assert(rem_set_name() == GenRemSet::CardTable, "unrecognized GenRemSet::Name");
case GenRemSet::CardTable: { return new CardTableRS(whole_heap, max_covered_regions);
CardTableRS* res = new CardTableRS(whole_heap, max_covered_regions);
return res;
}
default:
guarantee(false, "unrecognized GenRemSet::Name");
return NULL;
}
} }
void CollectorPolicy::cleared_all_soft_refs() { void CollectorPolicy::cleared_all_soft_refs() {

9
hotspot/src/share/vm/memory/genRemSet.cpp
View file

@ -32,13 +32,8 @@
// enumeration.) // enumeration.)
uintx GenRemSet::max_alignment_constraint(Name nm) { uintx GenRemSet::max_alignment_constraint(Name nm) {
switch (nm) { assert(nm == GenRemSet::CardTable, "Unrecognized GenRemSet type.");
case GenRemSet::CardTable: return CardTableRS::ct_max_alignment_constraint();
return CardTableRS::ct_max_alignment_constraint();
default:
guarantee(false, "Unrecognized GenRemSet type.");
return (0); // Make Windows compiler happy
}
} }
class HasAccumulatedModifiedOopsClosure : public KlassClosure { class HasAccumulatedModifiedOopsClosure : public KlassClosure {