Merge lp:~zorba-coders/zorba/disable_assert-invariant into lp:zorba
Proposed by
Matthias Brantner
Status: | Merged |
---|---|
Approved by: | Matthias Brantner |
Approved revision: | 11607 |
Merged at revision: | 11632 |
Proposed branch: | lp:~zorba-coders/zorba/disable_assert-invariant |
Merge into: | lp:zorba |
Diff against target: |
12 lines (+1/-1) 1 file modified
src/store/naive/json_items.h (+1/-1) |
To merge this branch: | bzr merge lp:~zorba-coders/zorba/disable_assert-invariant |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ghislain Fourny | Approve | ||
Matthias Brantner | Approve | ||
Review via email: mp+182971@code.launchpad.net |
Commit message
disable ASSERT_INVARIANT always (not only if NDEBUG is set). The problem with this
is that it depends on the CMake version whether NDEBUG is set in RelWithDebInfo.
This might cause unexpected slow queries with some CMake versions.
To post a comment you must log in.
Validation queue started for merge proposals: /code.launchpad .net/~zorba- coders/ zorba/disable_ assert- invariant/ +merge/ 182971
https:/
View workflow at http:// 74.93.6. 105:8888/ job/MP/ 47/