comparison src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp @ 12830:9b4d0569f2f4

8025852: Remove unnecessary setters in collector policy classes Summary: Use instance variables directly within the collector policy classes and remove unused setters. Reviewed-by: tschatzl, jcoomes
author jwilhelm
date Thu, 03 Oct 2013 21:36:29 +0200
parents 84683e78e713
children 24f32d09a0d7
comparison
equal deleted inserted replaced
12351:c49c7f835e8d 12830:9b4d0569f2f4
317 _collectionSetChooser = new CollectionSetChooser(); 317 _collectionSetChooser = new CollectionSetChooser();
318 _young_gen_sizer = new G1YoungGenSizer(); // Must be after call to initialize_flags 318 _young_gen_sizer = new G1YoungGenSizer(); // Must be after call to initialize_flags
319 } 319 }
320 320
321 void G1CollectorPolicy::initialize_flags() { 321 void G1CollectorPolicy::initialize_flags() {
322 set_min_alignment(HeapRegion::GrainBytes); 322 _min_alignment = HeapRegion::GrainBytes;
323 size_t card_table_alignment = GenRemSet::max_alignment_constraint(rem_set_name()); 323 size_t card_table_alignment = GenRemSet::max_alignment_constraint(rem_set_name());
324 size_t page_size = UseLargePages ? os::large_page_size() : os::vm_page_size(); 324 size_t page_size = UseLargePages ? os::large_page_size() : os::vm_page_size();
325 set_max_alignment(MAX3(card_table_alignment, min_alignment(), page_size)); 325 _max_alignment = MAX3(card_table_alignment, _min_alignment, page_size);
326 if (SurvivorRatio < 1) { 326 if (SurvivorRatio < 1) {
327 vm_exit_during_initialization("Invalid survivor ratio specified"); 327 vm_exit_during_initialization("Invalid survivor ratio specified");
328 } 328 }
329 CollectorPolicy::initialize_flags(); 329 CollectorPolicy::initialize_flags();
330 } 330 }