[kdevplatform] UI fix: add "Analyzers" category for appropriate plugins
ClosedPublic

Authored by antonanikin on Sep 13 2016, 11:07 AM.

Details

Summary

This patch add "Analyzers" category for plugins and groups its config pages in KDevelop's main config page. This provides more consistent user interface for set of appropriate plugins: cppcheck, vera++, valgrind, etc.

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
antonanikin retitled this revision from to [kdevplatform] UI fix: add "Analyzers" category for appropriate plugins.
antonanikin updated this object.
antonanikin edited the test plan for this revision. (Show Details)
antonanikin added a reviewer: KDevelop.
antonanikin set the repository for this revision to R33 KDevPlatform.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptSep 13 2016, 11:07 AM
kfunk accepted this revision.Sep 13 2016, 12:09 PM
kfunk added a reviewer: kfunk.
kfunk added a subscriber: kfunk.

Rest LGTM

shell/uicontroller.cpp
531 ↗(On Diff #6687)

I don't think we should move the background parser settings page to the Analyzers category. Thinking about it, it probably makes more sense to move it to the Language Support category.

This revision is now accepted and ready to land.Sep 13 2016, 12:09 PM
This revision was automatically updated to reflect the committed changes.
antonanikin marked an inline comment as done.Sep 13 2016, 2:25 PM
antonanikin added inline comments.
shell/uicontroller.cpp
531 ↗(On Diff #6687)

Ok, you right. Parser moved to "Language Support" category.