Unity

Merge lp://staging/~azzar1/unity/fix-1036231 into lp://staging/unity

  1. fix-1036231
  2. Merge into trunk
Proposed by Andrea Azzarone
Status: Merged
Approved by: Tim Penhey
Approved revision: no longer in the source branch.
Merged at revision: 2607
Proposed branch: lp://staging/~azzar1/unity/fix-1036231
Merge into: lp://staging/unity
Diff against target: 12 lines (+1/-1)
1 file modified
unity-shared/IconRenderer.cpp (+1/-1)
To merge this branch: bzr merge lp://staging/~azzar1/unity/fix-1036231
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Sam Spilsbury (community) Needs Information
Review via email: mp+120526@code.staging.launchpad.net

Commit message

Use linear filtering to draw icons (when needed).

To post a comment you must log in.
Revision history for this message
Sam Spilsbury (smspillaz) wrote :

LGTM. Is this code covered by tests? Should it be?

review: Needs Information
Revision history for this message
Andrea Azzarone (azzar1) wrote :

> LGTM. Is this code covered by tests? Should it be?

IconRenderer is test free :/
Btw this is a visual change and we don't need test for visual changes. I'll ask Tim btw.

Revision history for this message
Tim Penhey (thumper) wrote :

Just visual. OK to land.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.