D14931: Eliminate duplicate QMaps in OutputWidget
noreply at phabricator.kde.org
Sun Aug 19 19:37:50 BST 2018
vkorneev created this revision.
Herald added a project: KDevelop.
Herald added a subscriber: kdevelop-devel.
vkorneev requested review of this revision.
There were 3 QMaps with identical keys used for storing corresponding
view, proxy model and filter. In this version only one map is used - this
reduces the amount of map lookups and potential mistakes during code
This patch isn't supposed to change any behavior in OutputWidget, so we
need to assure that OutputWidget works as usually. For example:
create a few outputs;
enter the filters for some of the outputs;
choose random output;
close all other outputs;
check if the output is shown, filter corresponds to output, etc;
repeat previous actions a few times.
refactor_outputwidget (branched from master)
Cc: kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the KDevelop-devel