haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

COMMIT class: removed m_committed flag, added clear() and Empty() methods.

This commit is contained in:
Maciej Suminski 2016-06-15 10:15:10 +02:00
parent b815ea7865
commit 940765f4b3
2 changed files with 15 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
common/commit.cpp
View File

@ -27,13 +27,10 @@
COMMIT::COMMIT()
{
m_committed = false;
}
COMMIT::~COMMIT()
{
if( !m_committed )
{
for( COMMIT_LINE& ent : m_changes )
{
@ -41,7 +38,6 @@ COMMIT::~COMMIT()
delete ent.m_copy;
}
}
}
COMMIT& COMMIT::Stage( EDA_ITEM* aItem, CHANGE_TYPE aChangeType )

14
common/commit.h
View File

@ -89,6 +89,11 @@ public:
///> Revertes the commit by restoring the modifed items state.
virtual void Revert() = 0;
bool Empty() const
{
return m_changes.empty();
}
protected:
struct COMMIT_LINE
{
@ -97,14 +102,19 @@ protected:
CHANGE_TYPE m_type;
};
// Should be called in Push() & Revert() methods
void clear()
{
m_changedItems.clear();
m_changes.clear();
}
virtual void makeEntry( EDA_ITEM* aItem, CHANGE_TYPE aType, EDA_ITEM* aCopy = NULL );
virtual EDA_ITEM* parentObject( EDA_ITEM* aItem ) const = 0;
CHANGE_TYPE convert( UNDO_REDO_T aType ) const;
bool m_committed;
std::set<EDA_ITEM*> m_changedItems;
std::vector<COMMIT_LINE> m_changes;
};