Skip to content

Commit b8e8bab

Browse files
committed
Revert "Skip temporarily the assert in umf_ba_linear_destroy()"
This reverts commit 66ac900. Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
1 parent 99b1c19 commit b8e8bab

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

src/base_alloc/base_alloc_linear.c

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -251,14 +251,7 @@ void umf_ba_linear_destroy(umf_ba_linear_pool_t *pool) {
251251
if (pool->metadata.global_n_allocs) {
252252
fprintf(stderr, "umf_ba_linear_destroy(): global_n_allocs = %zu\n",
253253
pool->metadata.global_n_allocs);
254-
// This assert fails sporadically on Windows only.
255-
// TODO: fix this issue and uncomment this assert
256-
// It is safe to comment it out temporarily,
257-
// because the linear base allocator is used
258-
// in the proxy library only but umf_ba_linear_destroy()
259-
// is skipped, so this code is never run in real life.
260-
//
261-
// assert(pool->metadata.global_n_allocs == 0);
254+
assert(pool->metadata.global_n_allocs == 0);
262255
}
263256
#endif /* NDEBUG */
264257

0 commit comments

Comments
 (0)