Review Request: Do not emit KEditToolBar::newToolBarConfig twice

Commit Hook null at kde.org
Fri Jan 27 13:21:05 GMT 2012


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


This review has been submitted with commit e7ee12fcb0719f1cdd5b87df377944ea9ad20c3c by Dawit Alemayehu to branch KDE/4.8.

- Commit Hook


On Jan. 2, 2012, 6:25 p.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103579/
> -----------------------------------------------------------
> 
> (Updated Jan. 2, 2012, 6:25 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> The attached one liner patch stops KEditToolBar from emitting the newToolBarConfig signal when the user presses the OK button  without modifying anything after pressing Apply. IOW, the changes have already been applied.
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kedittoolbar.cpp ed572bf 
> 
> Diff: http://git.reviewboard.kde.org/r/103579/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120127/79d582b1/attachment.htm>


More information about the kde-core-devel mailing list