Review Request: Add options to StandardOutputView plugin
Silvère Lestang
silvere.lestang at gmail.com
Wed Feb 16 14:27:05 UTC 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100666/
-----------------------------------------------------------
Review request for KDevelop.
Summary
-------
Add to StandardOutputView plugin:
* An option to display or not the two buttons "Select activated Item" and "Focus when selecting Item"
* An optional filter action, added to "build", "run" and "debug" outputview
* the possibility to add a list of QAction to an outputview (the actions appears in the toolbar and the context menu)
Fix: now when we remove an output view from one Area, it's removed from all Areas
Know bug: If you remove quickly many outputs displayed in more than one Area, KDevelop crash. I can't fix the bug as I don't know where and why it's appears exactly (if I attached the debugger, KDevelop is slower so making the bug disappear).
Diffs
-----
outputview/ioutputview.h c889003
plugins/standardoutputview/CMakeLists.txt bdb23e4
plugins/standardoutputview/kdevstandardoutputview.rc 5455135
plugins/standardoutputview/outputwidget.h eef3844
plugins/standardoutputview/outputwidget.cpp 18e35e7
plugins/standardoutputview/standardoutputview.h 4f7e1bf
plugins/standardoutputview/standardoutputview.cpp fbdae75
plugins/standardoutputview/tests/CMakeLists.txt PRE-CREATION
plugins/standardoutputview/tests/standardoutputviewtest.h PRE-CREATION
plugins/standardoutputview/tests/standardoutputviewtest.cpp PRE-CREATION
plugins/standardoutputview/toolviewdata.h 14afd45
Diff: http://git.reviewboard.kde.org/r/100666/diff
Testing
-------
A test has been adding for the plugin - it's pass - but it's reimplement the macro QTEST_KDEMAIN to get round a bug that I don't understand.
Thanks,
Silvère
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110216/0a6c46f8/attachment.html>
More information about the KDevelop-devel
mailing list