Skip to content

Commit c781594

Browse files
committed
8253224: Shenandoah: ShenandoahStrDedupQueue destructor calls virtual num_queues()
Reviewed-by: rkennke, zgu
1 parent 33f8e70 commit c781594

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/hotspot/share/gc/shenandoah/shenandoahStrDedupQueue.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ ShenandoahStrDedupQueue::ShenandoahStrDedupQueue() :
5050

5151
ShenandoahStrDedupQueue::~ShenandoahStrDedupQueue() {
5252
MonitorLocker ml(StringDedupQueue_lock, Mutex::_no_safepoint_check_flag);
53-
for (size_t index = 0; index < num_queues(); index ++) {
53+
for (size_t index = 0; index < num_queues_nv(); index ++) {
5454
release_buffers(queue_at(index));
5555
}
5656

src/hotspot/share/gc/shenandoah/shenandoahStrDedupQueue.hpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,11 @@ class ShenandoahStrDedupQueue : public StringDedupQueue {
9999
void verify_impl();
100100

101101
protected:
102-
size_t num_queues() const { return (_num_producer_queue + 2); }
102+
size_t num_queues() const { return num_queues_nv(); }
103103

104104
private:
105+
inline size_t num_queues_nv() const { return (_num_producer_queue + 2); }
106+
105107
ShenandoahQueueBuffer* new_buffer();
106108

107109
void release_buffers(ShenandoahQueueBuffer* list);

0 commit comments

Comments
 (0)