From 3777bcbd87933515960cb0763d3120a7ae498a28 Mon Sep 17 00:00:00 2001 From: Ruben Dashyan Date: Fri, 20 Sep 2019 16:43:32 +0200 Subject: [PATCH] [apps/shared/expression_model_store] Remove redundant assert If the record is null, it means that the do-while loop has iterated too many times and there is no record left. However that situation is already asserted at the begining of the method. --- apps/shared/expression_model_store.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/apps/shared/expression_model_store.cpp b/apps/shared/expression_model_store.cpp index 00289df4e..686209a1e 100644 --- a/apps/shared/expression_model_store.cpp +++ b/apps/shared/expression_model_store.cpp @@ -63,10 +63,6 @@ Ion::Storage::Record ExpressionModelStore::recordSatisfyingTestAtIndex(int i, Mo Ion::Storage::Record record; do { record = recordAtIndex(index++); - if (record.isNull()) { - assert(false); - break; - } if (test(privateModelForRecord(record), context)) { if (i == count) { break;