The menu we retun can outlive the manager. So we can't use it or its members in
slots when connecting to signals of the menu or it's actions. This also switches
the actions' data() back to being the schemePaths instead of the rows of the
corresponding indices.
Details
Details
- Reviewers
broulik - Group Reviewers
Frameworks - Commits
- R265:a33675fdb3e4: Don't assume the manager and menu have the same lifetime
Open Labplot which currently triggers this crash.
Diff Detail
Diff Detail
- Repository
- R265 KConfigWidgets
- Branch
- menumanager (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 21525 Build 21543: arc lint + arc unit
src/kcolorschememanager.cpp | ||
---|---|---|
191 | You can connect to qApp |
src/kcolorschememanager.cpp | ||
---|---|---|
191 | What is meant by "connecting to qApp"? Just doing the scheme switching here? I wanted to avoid the duplication that's why I created the static helper method. |
src/kcolorschememanager.cpp | ||
---|---|---|
191 | Using qApp as context object |
src/kcolorschememanager.cpp | ||
---|---|---|
191 | Thanks! Makes sense! |