Skip to content

Commit

Permalink
Fix LevelDB build when asserts are enabled in release builds. (#367)
Browse files Browse the repository at this point in the history
* Fix LevelDB build when asserts are enabled in release builds.

BUG=https://bugs.chromium.org/p/chromium/issues/detail?id=603166

* fix

* Add comment
  • Loading branch information
jabdelmalek authored and cmumford committed Apr 15, 2016
1 parent ea992b4 commit a7bff69
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions table/format.cc
Original file line number Diff line number Diff line change
Expand Up @@ -30,15 +30,14 @@ Status BlockHandle::DecodeFrom(Slice* input) {
}

void Footer::EncodeTo(std::string* dst) const {
#ifndef NDEBUG
const size_t original_size = dst->size();
#endif
metaindex_handle_.EncodeTo(dst);
index_handle_.EncodeTo(dst);
dst->resize(2 * BlockHandle::kMaxEncodedLength); // Padding
PutFixed32(dst, static_cast<uint32_t>(kTableMagicNumber & 0xffffffffu));
PutFixed32(dst, static_cast<uint32_t>(kTableMagicNumber >> 32));
assert(dst->size() == original_size + kEncodedLength);
(void)original_size; // Disable unused variable warning.
}

Status Footer::DecodeFrom(Slice* input) {
Expand Down

0 comments on commit a7bff69

Please sign in to comment.