diff --git a/src/leveldbwrapper.cpp b/src/leveldbwrapper.cpp index 2a3e777c9..ace633c6a 100644 --- a/src/leveldbwrapper.cpp +++ b/src/leveldbwrapper.cpp @@ -95,7 +95,7 @@ bool CLevelDBWrapper::IsEmpty() it->SeekToFirst(); return !(it->Valid()); } - + CLevelDBIterator::~CLevelDBIterator() { delete piter; } bool CLevelDBIterator::Valid() { return piter->Valid(); } void CLevelDBIterator::SeekToFirst() { piter->SeekToFirst(); } diff --git a/src/leveldbwrapper.h b/src/leveldbwrapper.h index daa0f78ce..e422a6e01 100644 --- a/src/leveldbwrapper.h +++ b/src/leveldbwrapper.h @@ -67,7 +67,12 @@ private: leveldb::Iterator *piter; public: - CLevelDBIterator(leveldb::Iterator *piterIn) : piter(piterIn) {} + + /** + * @param[in] piterIn The original leveldb iterator. + */ + CLevelDBIterator(leveldb::Iterator *piterIn) : + piter(piterIn) { }; ~CLevelDBIterator(); bool Valid(); @@ -229,7 +234,7 @@ public: CLevelDBIterator *NewIterator() { return new CLevelDBIterator(pdb->NewIterator(iteroptions)); - { + } /** * Return true if the database managed by this class contains no entries. diff --git a/src/txdb.cpp b/src/txdb.cpp index 29b49745c..b416b1ad3 100644 --- a/src/txdb.cpp +++ b/src/txdb.cpp @@ -162,7 +162,7 @@ bool CCoinsViewDB::GetStats(CCoinsStats &stats) const { /* It seems that there are no "const iterators" for LevelDB. Since we only need read operations on it, use a const-cast to get around that restriction. */ - boost::scoped_ptr pcursor(const_cast(&db)->NewIterator()); + boost::scoped_ptr pcursor(const_cast(&db)->NewIterator()); pcursor->Seek('c'); CHashWriter ss(SER_GETHASH, PROTOCOL_VERSION); @@ -249,9 +249,9 @@ bool CBlockTreeDB::ReadFlag(const std::string &name, bool &fValue) { bool CBlockTreeDB::LoadBlockIndexGuts() { - boost::scoped_ptr pcursor(NewIterator()); + boost::scoped_ptr pcursor(NewIterator()); - pcursor->Seek(make_pair('b', uint256(0))); + pcursor->Seek(make_pair('b', uint256())); // Load mapBlockIndex while (pcursor->Valid()) {