IARC-CSU / CanReg5

CanReg5 is a multi user, multi platform, open source tool to input, store, check and analyse cancer registry data.
http://www.iacr.com.fr/CanReg5
GNU General Public License v3.0
24 stars 13 forks source link

C232: Avoid string concatenations in logs #118

Closed fbinfotel closed 2 years ago

fbinfotel commented 2 years ago

A placeholder is used most often, instead of building a String. In Production, the loggers are usually adjusted to a certain level, which means that the logs with a lower level will not be written: in the code, either the logger should not be called if not necessary, or the arguments passed to the logger should not have any cpu cost. Ignoring this rule can lead to performance issues.

fbinfotel commented 2 years ago

he branch is https://github.com/infotel4iarc/CanReg5/tree/feature/C232

Here is the point that was made:

fbinfotel commented 2 years ago

the branch was merged to the dev branch and some fix were added later