lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master b57ed4a 18/46: Remove deleted DatumSequenceEd


From: Greg Chicares
Subject: [lmi-commits] [lmi] master b57ed4a 18/46: Remove deleted DatumSequenceEditor copy-related methods
Date: Wed, 22 Jul 2020 11:05:12 -0400 (EDT)

branch: master
commit b57ed4af09734803123af911fb3118e0b0193281
Author: Vadim Zeitlin <vadim@tt-solutions.com>
Commit: Vadim Zeitlin <vadim@tt-solutions.com>

    Remove deleted DatumSequenceEditor copy-related methods
    
    There is no need to delete them explicitly, as they're already deleted
    in the base wxGridCellEditor class. Besides, doing it only in this class
    but not in the other specialized grid editors was inconsistent and it
    seems better to stop doing it here rather than also doing it elsewhere.
    
    No real changes.
---
 census_view.cpp | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/census_view.cpp b/census_view.cpp
index 92eeb6d..1a0a811 100644
--- a/census_view.cpp
+++ b/census_view.cpp
@@ -1009,12 +1009,6 @@ class DatumSequenceEditor
   public:
     DatumSequenceEditor() = default;
 
-    // We don't define a copy ctor because wxGridCellEditor doesn't provide one
-    // and its derived classes are supposed to override Clone() to support
-    // polymorphic copying, instead of using the copy ctor.
-    DatumSequenceEditor(DatumSequenceEditor const&) = delete;
-    DatumSequenceEditor& operator=(DatumSequenceEditor const&) = delete;
-
     void Create(wxWindow* parent, wxWindowID id, wxEvtHandler* evtHandler) 
override;
 
     void BeginEdit(int row, int col, wxGrid* grid) override;



reply via email to

[Prev in Thread] Current Thread [Next in Thread]