commontk / CTK

A set of common support code for medical imaging, surgical navigation, and related purposes.
https://commontk.org
Apache License 2.0
855 stars 490 forks source link

Update ctkLogger #1181

Closed jcfr closed 8 months ago

jcfr commented 8 months ago
jcfr commented 8 months ago

@Punzo @lassoan There changes were originally part of https://github.com/commontk/CTK/pull/1165 and have been factored out into an independent pull request.

jcfr commented 8 months ago

Now that we removed support for Qt4, for new classes, I think we should instead standardize on using QLoggingCategory and avoid ctkLogger.

See https://doc.qt.io/qt-5/qloggingcategory.html

Punzo commented 8 months ago

@Punzo @lassoan There changes were originally part of #1165 and have been factored out into an independent pull request.

ok, perfect!

jcfr commented 8 months ago

I revisited the default level associated with the ctkLogger functions and also added a multiple tests.