diff --git a/src/gui/nfgheadergrid.cc b/src/gui/nfgheadergrid.cc index fd289dbe7..9cd18f729 100644 --- a/src/gui/nfgheadergrid.cc +++ b/src/gui/nfgheadergrid.cc @@ -92,15 +92,6 @@ class RowPlayerTable final : public wxGridTableBase { explicit RowPlayerTable(TableWidget *p_table) : m_table(p_table), m_renderer(new RowHeaderCellRenderer(p_table)) { - // Freshly-constructed wxGridCellWorker-derived objects start unreferenced; - // this is our own table's stake, matched by DecRef() in the destructor. - // Each SetRenderer() call below additionally consumes one reference per - // call (that's the ownership contract of that setter), so GetAttr() takes - // a fresh IncRef() immediately before each one -- the instance itself is - // reused across all cells and all calls, which is required: wxGrid - // expects the *same* renderer object identity back on repeated queries - // for a given cell. - m_renderer->IncRef(); } ~RowPlayerTable() override { m_renderer->DecRef(); } @@ -638,7 +629,6 @@ class ColPlayerTable final : public wxGridTableBase { explicit ColPlayerTable(TableWidget *p_table) : m_table(p_table), m_renderer(new ColHeaderCellRenderer(p_table)) { - m_renderer->IncRef(); } ~ColPlayerTable() override { m_renderer->DecRef(); } diff --git a/src/gui/nfgpayoffgrid.cc b/src/gui/nfgpayoffgrid.cc index ea3e70fd5..5465dad3f 100644 --- a/src/gui/nfgpayoffgrid.cc +++ b/src/gui/nfgpayoffgrid.cc @@ -135,8 +135,6 @@ class PayoffTable final : public wxGridTableBase { : m_table(p_table), m_renderer(new PayoffCellRenderer(p_table)), m_editor(new RationalCellEditor()) { - m_renderer->IncRef(); - m_editor->IncRef(); } ~PayoffTable() override