diff options
| author |  rsgowman <rgowman@google.com> | 2018-03-12 16:35:40 -0400 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2018-03-12 16:35:40 -0400 |
| commit | 3e41f2585cefaef1d37f76224cdd8e8908c529da (patch) | |
| tree | 7c5e2e498d3a5c4062c7ae5d02fcf695fc2d263a /Firestore/core/src/firebase/firestore/remote | |
| parent | f122cf7ce802972bd2fea45acac3deae2affcafa (diff) | |
Fix test failures that occur during prod build (#910)
* Fix ArraySortedMap.ChecksSize test (when NDEBUG is enabled)
assert() is a noop when NDEBUG is defined (i.e. during production
builds) so the size assertion check doesn't occur. Assuming we want this
check even during prod use, I've switched assert to FIREBASE_ASSERT.
* s/FIREBASE_ASSERT/FIREBASE_DEV_ASSERT/g in ordered_code.cc
This keeps the behaviour in line with the original (which purposely
differs if NDEBUG is defined or not.) Discovered due to the test suite
(correctly) breaking during a prod build.
Diffstat (limited to 'Firestore/core/src/firebase/firestore/remote')
0 files changed, 0 insertions, 0 deletions