Change NDEBUG to verbose logging (!92) · Merge requests · Knot projects / Knot Resolver · GitLab

Snippets Groups Projects

GitLab now enforces expiry dates on tokens that originally had no set expiration date. Those tokens were given an expiration date of one year later. Please review your personal access tokens, project access tokens, and group access tokens to ensure you are aware of upcoming expirations. Administrators of GitLab can find more information on how to identify and mitigate interruption in our documentation.

Self sign-up has been disabled due to increased spam activity. If you want to get access, please send an email to a project owner (preferred) or at gitlab(at)nic(dot)cz. We apologize for the inconvenience.

Merged Change NDEBUG to verbose logging

verboselog into master

Merged Ondřej Surý requested to merge verboselog into master 8 years ago

As requested in !90 (closed), I have renamed all debug and DEBUG to verbose (where it made sense), and renamed NLOGDEBUG to NOVERBOSELOG.

The logic is simple - most usually we want to have verbose logging compiled in and assert() (-DNDEBUG) not compiled in.

Activity

Ondřej Surý mentioned in merge request !90 (closed) 8 years ago

mentioned in merge request !90 (closed)
Vladimír Čunát added 1 commit 8 years ago
added 1 commit

2b16ea2c - rename: kr_verbose_do_log -> kr_verbose_status

Compare with previous version
Vladimír Čunát @vcunat · 8 years ago

Owner

The largest downside seems the potential for conflicts with unmerged branches etc., but they should be very simple to resolve.
Ondřej Surý @ondrej · 8 years ago

Author Guest

Yup, one more reason we should be aiming to release more often...
Vladimír Čunát merged 8 years ago

merged
Vladimír Čunát mentioned in commit e005f977 8 years ago

mentioned in commit e005f977

Please register or sign in to reply