Skip to content
Browse files

Fix LevelDB build when asserts are enabled in release builds. (#367)

* 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...
1 parent ea992b4 commit a7bff697baa062c8f6b8fb760eacf658712b611a @jabdelmalek jabdelmalek committed with cmumford Apr 15, 2016
Showing with 1 addition and 2 deletions.
  1. +1 −2 table/format.cc
View
3 table/format.cc
@@ -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) {

0 comments on commit a7bff69

Please sign in to comment.
Something went wrong with that request. Please try again.