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
Branch
Plasma/5.13
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 106
Build 106: arc lint + arc unit
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 36304

View Options

kcms/mouse/backends/x11/x11_backend.cpp

Loading...