QtMir

Merge lp:~aacid/qtmir/fix_debug_line into lp:qtmir

Proposed by Albert Astals Cid on 2015-04-21

Status:	Merged
Approved by:	Gerry Boland on 2015-04-21
Approved revision:	329
Merged at revision:	334
Proposed branch:	lp:~aacid/qtmir/fix_debug_line
Merge into:	lp:qtmir
Diff against target:	12 lines (+1/-1) 1 file modified src/modules/Unity/Application/application_manager.cpp (+1/-1)
To merge this branch:	bzr merge lp:~aacid/qtmir/fix_debug_line
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Gerry Boland (community)		2015-04-21	Approve on 2015-04-21
Review via email: mp+256905@code.launchpad.net

Fix debug line

* 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?
Yes

* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

Revision history for this message

Gerry Boland (gerboland) on 2015-04-21:

review: Approve

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

People subscribed via source and target branches

to all changes:

1	=== modified file 'src/modules/Unity/Application/application_manager.cpp'
2	--- src/modules/Unity/Application/application_manager.cpp 2015-02-25 11:27:56 +0000
3	+++ src/modules/Unity/Application/application_manager.cpp 2015-04-21 10:28:07 +0000
4	@@ -706,7 +706,7 @@
5	QModelIndex appIndex = findIndex(application);
6	Q_EMIT dataChanged(appIndex, appIndex, QVector<int>() << role);
7
8	- qCDebug(QTMIR_APPLICATIONS) << "ApplicationManager::onAppDataChanged: Received " << m_roleNames[role] << " update", application->appId();
9	+ qCDebug(QTMIR_APPLICATIONS) << "ApplicationManager::onAppDataChanged: Received " << m_roleNames[role] << " update" << application->appId();
10	} else {
11	qCDebug(QTMIR_APPLICATIONS) << "ApplicationManager::onAppDataChanged: Received " << m_roleNames[role] << " signal but application has disappeard.";
12	}