Merge lp:~townsend/unity/fix-lp926979-trusty into lp:unity/7.2

Proposed by Christopher Townsend
Status: Merged
Approved by: Stephen M. Webb
Approved revision: no longer in the source branch.
Merged at revision: 3826
Proposed branch: lp:~townsend/unity/fix-lp926979-trusty
Merge into: lp:unity/7.2
Diff against target: 14 lines (+3/-1)
1 file modified
dash/DashView.cpp (+3/-1)
To merge this branch: bzr merge lp:~townsend/unity/fix-lp926979-trusty
Reviewer Review Type Date Requested Status
Stephen M. Webb (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+259288@code.launchpad.net

Commit message

Make sure scope_views_[filter.id] exists before using operator[] on it.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Stephen M. Webb (bregma) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'dash/DashView.cpp'
2--- dash/DashView.cpp 2014-07-24 14:03:10 +0000
3+++ dash/DashView.cpp 2015-05-15 19:53:47 +0000
4@@ -1174,7 +1174,9 @@
5
6 if (!filter.filters.empty())
7 {
8- scope_views_[filter.id]->filters_expanded = true;
9+ if (scope_views_.find(filter.id) != std::end(scope_views_))
10+ scope_views_[filter.id]->filters_expanded = true;
11+
12 // update the scope for each filter
13 for (auto p : filter.filters) {
14 UpdateScopeFilter(filter.id, p.first, p.second);

Subscribers

People subscribed via source and target branches

to status/vote changes: