cheng/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove unused wxGridCellChoiceEditor::PaintBackground() override

Browse Source 
This overridden method didn't do anything except calling the base class
version, so just remove it.
This commit is contained in:
Vadim Zeitlin 2020-06-22 03:05:52 +02:00
parent dddcdf62df
commit 32edcde5bd
2 changed files with 0 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
include/wx/generic/grideditors.h
View File

@ -307,10 +307,6 @@ public:
virtual void SetSize(const wxRect& rect) wxOVERRIDE;
virtual void PaintBackground(wxDC& dc,
const wxRect& rectCell,
const wxGridCellAttr& attr) wxOVERRIDE;
virtual void BeginEdit(int row, int col, wxGrid* grid) wxOVERRIDE;
virtual bool EndEdit(int row, int col, const wxGrid* grid,
const wxString& oldval, wxString *newval) wxOVERRIDE;

13
src/generic/grideditors.cpp
View File

@ -1459,19 +1459,6 @@ void wxGridCellChoiceEditor::SetSize(const wxRect& rect)
wxGridCellEditor::SetSize(rectChoice.CenterIn(rect, wxVERTICAL));
}
void wxGridCellChoiceEditor::PaintBackground(wxDC& dc,
const wxRect& rectCell,
const wxGridCellAttr& attr)
{
// as we fill the entire client area, don't do anything here to minimize
// flicker
// TODO: It doesn't actually fill the client area since the height of a
// combo always defaults to the standard. Until someone has time to
// figure out the right rectangle to paint, just do it the normal way.
wxGridCellEditor::PaintBackground(dc, rectCell, attr);
}
void wxGridCellChoiceEditor::BeginEdit(int row, int col, wxGrid* grid)
{
wxASSERT_MSG(m_control,