The Process class uses the pimpl pattern when it is not needed
Bug #1251183 reported by
Manuel de la Peña
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ubuntu-download-manager |
Fix Released
|
Medium
|
Manuel de la Peña | ||
ubuntu-download-manager (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
The Process class is using the pimpl pattern when it is not exposed to any other project and is just a wrapper class added for testing.
Related branches
lp:~mandel/ubuntu-download-manager/remove-process-pimpl
- PS Jenkins bot: Approve (continuous-integration)
- Diego Sarmentero (community): Approve
-
Diff: 147 lines (+22/-69)2 files modifiedlibubuntudownloadmanager/system/process.cpp (+19/-63)
libubuntudownloadmanager/system/process.h (+3/-6)
tags: |
added: tech-debt removed: tect-debt |
Changed in ubuntu-download-manager: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix committed into lp:ubuntu-download-manager at revision 194, scheduled for release in ubuntu- download- manager, milestone 0.3