Resolve "replace all C assert with PAssert" (!419) · Merge requests · OPAL / src · GitLab

Code indexing in gitaly is broken and leads to code not being visible to the user. We work on the issue with highest priority.

Snippets Groups Projects

Merged snuverink_j requested to merge 589-replace-all-c-assert-with-passert into master 4 years ago

All threads resolved!

This MR:

replaces C assert with PAssert (Closes #589 (closed))
cleans up PartBins and PartBinsCyc classes (and adds license header)

Edited 4 years ago by snuverink_j

Activity

snuverink_j changed milestone to %OPAL 2021.1 4 years ago

changed milestone to %OPAL 2021.1
snuverink_j added Enhancement label 4 years ago

added Enhancement label
snuverink_j changed the description 4 years ago

changed the description
snuverink_j added Cleanup label 4 years ago

added Cleanup label
frey_m @frey_m · 4 years ago

Maintainer

Resolved 4 years ago by frey_m

@snuverink_j You touched a few files that do not have the new file header. I suggest updating them.

Last reply by frey_m 4 years ago
frey_m @frey_m started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by snuverink_j
Last reply by snuverink_j 4 years ago

kraus @kraus started a thread on the diff 4 years ago

Resolved 4 years ago by kraus

snuverink_j added 1 commit 4 years ago

added 1 commit

54c163ba - replace BL_ASSERT with PAssert

Compare with previous version

kraus approved this merge request 4 years ago

approved this merge request

frey_m resolved all threads 4 years ago

resolved all threads

frey_m approved this merge request 4 years ago

approved this merge request

snuverink_j merged 4 years ago

merged

snuverink_j mentioned in commit 4 years ago

mentioned in commit a1d2c53b

Please register or sign in to reply