Differential D13599

Ref KConfig whilst we're using it
ClosedPublic
Actions

Authored by davidedmundson on Jun 19 2018, 8:02 AM.

Details

Reviewers
fvogt
Group Reviewers
Plasma
Commits
R119:ead2c3e859dc: Ref KConfig whilst we're using it
Summary

In current code we would have a KConfigGroup with a dangling KConfig
deleted after the RHS for the group fetch has finished.

BUG: 394534

Test Plan

Wrote minimal test case of code
It produced a valgrind warning (weirdly didn't crash though)
Modified to correct version
No longer any warnings

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Jun 19 2018, 8:02 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 19 2018, 8:02 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Jun 19 2018, 8:02 AM
fvogt accepted this revision.Jun 19 2018, 8:08 AM
This revision is now accepted and ready to land.Jun 19 2018, 8:08 AM
Closed by commit R119:ead2c3e859dc: Ref KConfig whilst we're using it (authored by davidedmundson). · Explain WhyJun 19 2018, 8:09 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 36305

View Options

kcms/mouse/backends/x11/x11_backend.cpp

Loading...