This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103323/ 

This review has been submitted with commit 2ca961213a86c58b113b1d955000fcb93b96625d by Jaime Torres to branch KDE/4.7.

- Commit


On December 4th, 2011, 10:17 a.m., Jaime Torres Amate wrote:

Review request for kdelibs and Aaron J. Seigo.
By Jaime Torres Amate.

Updated Dec. 4, 2011, 10:17 a.m.

Description 

In QString AssociatedApplicationManager::application it was returning a value and then trying to connect a signal.
If my change is wrong, there are two possible things to do:
* write a comment to explain the dead code
* remove the dead code

Diffs

  • plasma/widgets/signalplotter.cpp (aa132dd)
  • plasma/popupapplet.cpp (c363503)
  • plasma/private/associatedapplicationmanager.cpp (cf8bbe5)
  • plasma/private/style.cpp (e3449f5)

View Diff