Review Request 111453: move kacceleratormanager to KWidgetAddons

Wojciech Kapuscinski wojtask9.kde at gmail.com
Mon Jul 8 16:39:55 UTC 2013


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

Review request for KDE Frameworks.


Description
-------

move kacceleratormanager to KWidgetAddons.

Note:
This is only for review because this patch contains some ugly hack (export symbol from private class) and this patch is only Proof-on-concept.

Changes:
rename kacceleratormanager_private.h -> kacceleratormanager_p.h
move KAcceleratorManagerPrivate class definition from kacceleratormanager.cpp to kacceleratormanager_p.h
set translated texts when KStandardAction is constructed for the firs time.

Open questions:
- should I add static method for example initStandardNames in KAcceleratorManager namespace that replace appendStandarNames and remove export symbol from KAcceleratorManagerPrivate ?
- use QMutex in KAcceleratorManager that guarantees thread-safety (lock/unlock mutex in standardNames and initStandardNames)?


Diffs
-----

  kdeui/CMakeLists.txt 35b4019 
  kdeui/shortcuts/kacceleratormanager.h 68e87d2 
  kdeui/shortcuts/kacceleratormanager.cpp 990b093 
  kdeui/shortcuts/kacceleratormanager_private.h ab04d42 
  staging/kwidgets/src/actions/kstandardaction.cpp 0c3733a 
  tier1/kwidgetsaddons/src/CMakeLists.txt 82d42db 

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


Testing
-------


Thanks,

Wojciech Kapuscinski

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130708/f2bdd1e1/attachment.html>


More information about the Kde-frameworks-devel mailing list