Skip to content

Commit c781594

Browse files
committedSep 16, 2020
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

+1-1
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

+3-1
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)
Please sign in to comment.