Differential D3993

disable logging to kscreen.log by default
ClosedPublic
Actions

Authored by sebas on Jan 6 2017, 4:48 PM.

Details

Reviewers
mart
Group Reviewers
Plasma
Commits
R110:cea4c63f8465: disable logging to kscreen.log by default
Summary

Most of the issues that this feature should uncover have been uncovered,
it makes sense to disable the logging (and futzing around with
QCategorizedLogging) by default, and only use it to debug specific
setups.

Test Plan

Adjusted autotests accordingly.

Diff Detail

Repository
R110 KScreen Library
Branch
sebas/disablelogging
Lint
No Linters Available
Unit
No Unit Test Coverage
sebas updated this revision to Diff 9804.Jan 6 2017, 4:48 PM
sebas retitled this revision from to disable logging to kscreen.log by default.
sebas updated this object.
sebas edited the test plan for this revision. (Show Details)
sebas added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptJan 6 2017, 4:48 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
graesslin added a subscriber: graesslin.Jan 6 2017, 6:33 PM

+1

autotests/testlog.cpp
93

typo

mart accepted this revision.Jan 10 2017, 3:47 PM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Jan 10 2017, 3:47 PM
Closed by commit R110:cea4c63f8465: disable logging to kscreen.log by default (authored by sebas). · Explain WhyJan 11 2017, 10:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mautotests/testlog.cpp (8 lines)
Msrc/log.cpp (10 lines)

Diff 9804

View Options

autotests/testlog.cpp

Loading...
View Options

src/log.cpp

Loading...