diff options
author | Matthias Beyer <mail@beyermatthias.de> | 2019-06-26 18:56:35 +0200 |
---|---|---|
committer | Matthias Beyer <mail@beyermatthias.de> | 2019-08-05 20:09:07 +0200 |
commit | 38ec9d7127921725dbb6e9e1429bb3e3e54a814e (patch) | |
tree | 6f45dbc5156e568db40a89b72ea60ae4f589fbf3 | |
parent | 882c9f9122f102ed91403f47345e8c95e5cc41c5 (diff) |
Add debugging output as task
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
-rw-r--r-- | content/blog/2018-12-14-call-for-participation-3.md | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/content/blog/2018-12-14-call-for-participation-3.md b/content/blog/2018-12-14-call-for-participation-3.md index ad65797..9840444 100644 --- a/content/blog/2018-12-14-call-for-participation-3.md +++ b/content/blog/2018-12-14-call-for-participation-3.md @@ -27,6 +27,20 @@ Contributions (patches, questions, issues/bug reports) can be filed via the <hr> +## Debugging output + +**Complexity: low, Effort: low** + +The imag codebase contains a lot of `debug!()` statements. But it is not enough. +Sometimes even, the codebase has `debug!()` statements where it should have +`trace!()` statements. + +The general rule is: Use `debug!()` for telling what is done, use `trace!()` for +show details, variables, what exact functions are called, etc. + +There cannot be enough `trace!()` statements in the codebase! + + ## Replace is-match with matches **Complexity: low, Effort: low** |