Merge lp:~dandrader/qtmir/improveSessionDebugLogging into lp:qtmir
Proposed by
Daniel d'Andrada
Status: | Merged |
---|---|
Approved by: | Nick Dedekind |
Approved revision: | no longer in the source branch. |
Merged at revision: | 517 |
Proposed branch: | lp:~dandrader/qtmir/improveSessionDebugLogging |
Merge into: | lp:qtmir |
Diff against target: |
233 lines (+44/-22) 3 files modified
src/modules/Unity/Application/mirsurfacelistmodel.cpp (+21/-0) src/modules/Unity/Application/mirsurfacelistmodel.h (+2/-0) src/modules/Unity/Application/session.cpp (+21/-22) |
To merge this branch: | bzr merge lp:~dandrader/qtmir/improveSessionDebugLogging |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Nick Dedekind (community) | Approve | ||
Unity8 CI Bot (community) | continuous-integration | Approve | |
Review via email: mp+296199@code.launchpad.net |
Commit message
Improve Session debug logging
Description of the change
* Are there any related MPs required for this MP to build/function as expected? Please list.
No, but it goes well along with https:/
* Did you perform an exploratory manual test run of your code change and any related functionality?
Yes
* 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.
On 01/06/2016 09:42, Nick Dedekind wrote: :MirSurfaceList Interface &surfaceListConst)
>> +QDebug operator<<(QDebug dbg, const unityapp:
> I don't see this actually being used anywhere.
>
It is not. This is just a facility to help with debugging when you add
ad-hoc, temporary, qDebug() entries to investigate something.
So you can just do "qDebug() << someMirSurfaceList" and its contents
will be nicely printed.
Similar to what most Qt classes have (like QPoint, QRect etc).