From 655e29ff1a06509a8f6b53d74980627f307b34b0 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sun, 19 Jan 2020 03:09:29 +0100 Subject: [PATCH] Remove a duplicate OnColumnChange() call If the width really changed, wxDataViewColumn::SetWidth() already calls wxDataViewCtrl::OnColumnChange() (via UpdateWidth()), so it's unnecessary to call the latter again immediately after calling the former. And if the width didn't change, it's not necessary to call it at all, so in either case the OnColumnChange() call can be just removed. No real changes. --- src/generic/datavgen.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/src/generic/datavgen.cpp b/src/generic/datavgen.cpp index 762abf92ba..1ce4bb53ae 100644 --- a/src/generic/datavgen.cpp +++ b/src/generic/datavgen.cpp @@ -402,7 +402,6 @@ private: const unsigned col = event.GetColumn(); owner->GetColumn(col)->SetWidth(event.GetWidth()); - GetOwner()->OnColumnChange(col); } void OnEndReorder(wxHeaderCtrlEvent& event)