Add feature to KDevelop::ProblemModel for show/hide problems source
ClosedPublic

Authored by antonanikin on Feb 28 2017, 2:41 AM.

Details

Summary

The patch goal is providing simple mechanism to show/hide problems source in the Problems View. Most of current problems detectors (cppcheck, vera++, clang-tidy) produces problems which have only one type of source. This leads to displaying "Source" column with single value which doesn't have any sense and produce unneeded screen space usage. Therefore by default this feature is disabled and enabled only for models with different-sources problems like standard KDevelop::ProblemReporterModel (lexical/syntax/to-do/etc parsing).

ProblemReporter, NEW=OLD behavior (source is displayed):

Cppcheck, OLD behavior (source is displayed):

Cppcheck, NEW behavior (source is not displayed):

Test Plan

Tested on master branch

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 created this revision.Feb 28 2017, 2:41 AM
antonanikin edited the summary of this revision. (Show Details)Feb 28 2017, 2:55 AM
antonanikin edited the test plan for this revision. (Show Details)
kfunk accepted this revision.Feb 28 2017, 7:27 AM
kfunk added a subscriber: kfunk.

Yep, this makes a lot of sense.

Thanks for streamlining all this!

This revision is now accepted and ready to land.Feb 28 2017, 7:27 AM
This revision was automatically updated to reflect the committed changes.