Skip to content
Permalink
Browse files
8240133: G1DirtyCardQueue destructor has useless flush
Removed useless call to flush.

Reviewed-by: tschatzl, sjohanss
  • Loading branch information
Kim Barrett committed Mar 8, 2020
1 parent 3d9dddd commit 7806ca14773fd3c7ee895a1664d9daa6483822a5
Showing with 0 additions and 8 deletions.
  1. +0 −4 src/hotspot/share/gc/g1/g1DirtyCardQueue.cpp
  2. +0 −4 src/hotspot/share/gc/g1/g1DirtyCardQueue.hpp
@@ -50,10 +50,6 @@ G1DirtyCardQueue::G1DirtyCardQueue(G1DirtyCardQueueSet* qset) :
PtrQueue(qset, true /* active */)
{ }

G1DirtyCardQueue::~G1DirtyCardQueue() {
flush();
}

void G1DirtyCardQueue::handle_completed_buffer() {
assert(_buf != NULL, "precondition");
BufferNode* node = BufferNode::make_node_from_buffer(_buf, index());
@@ -44,10 +44,6 @@ class G1DirtyCardQueue: public PtrQueue {
public:
G1DirtyCardQueue(G1DirtyCardQueueSet* qset);

// Flush before destroying; queue may be used to capture pending work while
// doing something else, with auto-flush on completion.
~G1DirtyCardQueue();

// Process queue entries and release resources.
void flush() { flush_impl(); }

0 comments on commit 7806ca1

Please sign in to comment.