Unity

Merge lp:~jincreator/unity/lp1251193 into lp:unity

  1. lp1251193
  2. Merge into trunk
Proposed by Jinkyu Yi on 2014-05-03
Status: Merged
Approved by: Marco Trevisan (Treviño) on 2014-05-05
Approved revision: 3795
Merged at revision: 3796
Proposed branch: lp:~jincreator/unity/lp1251193
Merge into: lp:unity
Diff against target: 12 lines (+1/-1)
1 file modified
dash/ResultRendererTile.cpp (+1/-1)
To merge this branch: bzr merge lp:~jincreator/unity/lp1251193
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) 2014-05-03 Approve on 2014-05-05
Review via email: mp+218194@code.launchpad.net

Commit message

Dash: Fix problem at checking blacklisted unicode range

To post a comment you must log in.
Marco Trevisan (Treviño) (3v1n0) wrote :

Thanks, looks sane to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'dash/ResultRendererTile.cpp'
--- dash/ResultRendererTile.cpp 2014-03-20 05:05:21 +0000
+++ dash/ResultRendererTile.cpp 2014-05-03 18:20:48 +0000
@@ -424,7 +424,7 @@
424bool IsBlacklistedChar(gunichar uni_c)424bool IsBlacklistedChar(gunichar uni_c)
425{425{
426 if ((uni_c >= 0x1000 && uni_c <= 0x109F) ||426 if ((uni_c >= 0x1000 && uni_c <= 0x109F) ||
427 (uni_c >= 0xAA60 && uni_c >= 0xAA7B))427 (uni_c >= 0xAA60 && uni_c <= 0xAA7B))
428 {428 {
429 return true;429 return true;
430 }430 }