Depends on D27127
Details
Details
- Reviewers
ervin crossi bport - Group Reviewers
Plasma - Commits
- R119:ca02af6584c6: Solid-device-automounter/kcm: Get rid of singleton for AutomounterSettings
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Branch
- arcpatch-D27127
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 22635 Build 22653: arc lint + arc unit
solid-device-automounter/kded/DeviceAutomounter.cpp | ||
---|---|---|
39 | m_settings need to go to the next line |
solid-device-automounter/kcm/DeviceAutomounterKCM.cpp | ||
---|---|---|
44 | please break the line before ", m_devices" | |
solid-device-automounter/kcm/DeviceModel.cpp | ||
34 | m_settings on its own line? | |
solid-device-automounter/kcm/DeviceModel.h | ||
37 | parent should come last and keep the = nullptr | |
solid-device-automounter/kded/DeviceAutomounter.cpp | ||
111 | This is rather odd. A too eager search and replace? ;-) | |
solid-device-automounter/lib/AutomounterSettings.cpp | ||
23 | I guess you could have used using AutomounterSettingsBase::AutomounterSettingsBase; in the header instead. | |
solid-device-automounter/lib/AutomounterSettings.h | ||
31 | No space between the parenthesis |