Merge lp:~aacid/qtmir/fix_mock_constructor into lp:qtmir
Proposed by
Albert Astals Cid
Status: | Merged |
---|---|
Approved by: | Gerry Boland |
Approved revision: | 497 |
Merged at revision: | 513 |
Proposed branch: | lp:~aacid/qtmir/fix_mock_constructor |
Merge into: | lp:qtmir |
Diff against target: |
10 lines (+1/-0) 1 file modified
tests/framework/mock_mir_session.cpp (+1/-0) |
To merge this branch: | bzr merge lp:~aacid/qtmir/fix_mock_constructor |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Unity8 CI Bot (community) | continuous-integration | Approve | |
Gerry Boland (community) | Approve | ||
Review via email: mp+295910@code.launchpad.net |
Commit message
Initialize m_sessionId
It seems we could as well remove the constructor since it compiles fine without it, but this seems less intrusive
Description of the change
* Are there any related MPs required for this MP to build/function as expected?
No
* Did you perform an exploratory manual test run of your code change and any related functionality?
Not really
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
To post a comment you must log in.
This ok, good to init stuff instead of risking junk