Merge lp://qastaging/~thumper/unity/no-lambda-for-bug-964897 into lp://qastaging/unity
Proposed by
Tim Penhey
| Status: | Merged |
|---|---|
| Approved by: | Omer Akram |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 2614 |
| Proposed branch: | lp://qastaging/~thumper/unity/no-lambda-for-bug-964897 |
| Merge into: | lp://qastaging/unity |
| Diff against target: |
77 lines (+25/-20) 2 files modified
launcher/HudLauncherIcon.cpp (+24/-20) launcher/HudLauncherIcon.h (+1/-0) |
| To merge this branch: | bzr merge lp://qastaging/~thumper/unity/no-lambda-for-bug-964897 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Andrea Azzarone (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Changed code to get a better, readable stack trace for bug 964897
Description of the change
I'm at a bit of a loss as to how this bug is raising itself.
The lambda doesn't help in tracking this down, so I've extracted the method in the hope that should this happen again, we'll get a better stack trace.
That is all.
To post a comment you must log in.
Revision history for this message
| Unity Merger (unity-merger) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
LGTM.