Review Request: Avoid multiple map - engine connections

Jacopo De Simoi wilderkde at gmail.com
Mon May 21 13:51:58 UTC 2012


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

(Updated May 21, 2012, 1:51 p.m.)


Review request for Plasma and Aaron J. Seigo.


Description
-------

This fixes some multiple connection issues; up to now we connect the map to the engine once for each device that is added, hence when some device changes its state, it triggers the corresponding signal in the map, which in turn triggers multiple times  the corresponding slot in the engine.
 
This seems to have been like that from ages. Was there a reason for that?


Diffs
-----

  plasma/generic/dataengines/soliddevice/devicesignalmapmanager.cpp 4549cd4 

Diff: http://git.reviewboard.kde.org/r/105003/diff/


Testing
-------

basic testing: everything works as expected.


Thanks,

Jacopo De Simoi

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120521/10113e7e/attachment.html>


More information about the Plasma-devel mailing list