Unity Videos Lens

Merge lp:~davidc3/unity-lens-video/fixes-1062037 into lp:unity-lens-video

  1. fixes-1062037
  2. Merge into trunk
Proposed by David Callé on 2012-10-10
Status: Merged
Approved by: Łukasz Zemczak on 2012-10-10
Approved revision: 98
Merged at revision: 98
Proposed branch: lp:~davidc3/unity-lens-video/fixes-1062037
Merge into: lp:unity-lens-video
Diff against target: 12 lines (+1/-1)
1 file modified
src/unity-lens-video (+1/-1)
To merge this branch: bzr merge lp:~davidc3/unity-lens-video/fixes-1062037
Reviewer Review Type Date Requested Status
Łukasz Zemczak 2012-10-10 Approve on 2012-10-10
Review via email: mp+128893@code.launchpad.net

Commit Message

Don't try to encode strings returned by Gio, they already are utf-8

Description of the Change

Don't try to encode strings returned by Gio, they already are utf-8. Only strings returned by Zeitgeist need it.

To post a comment you must log in.
Łukasz Zemczak (sil2100) wrote :

Fixes the issue for me - and looks good. +1

review: Approve
Unity Merger (unity-merger) wrote :

No commit message specified.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/unity-lens-video'
2--- src/unity-lens-video 2012-10-04 17:13:31 +0000
3+++ src/unity-lens-video 2012-10-10 09:43:22 +0000
4@@ -482,7 +482,7 @@
5 g_file = Gio.file_new_for_uri(uri)
6 path = g_file.get_path()
7 if self.is_video(path) and not self.is_hidden(path, blacklist):
8- item.append(self.get_name(path).encode("utf-8"))
9+ item.append(self.get_name(path))
10 item.append('')
11 item.append(uri.encode("utf-8"))
12 item.append(self.get_icon(path))

Subscribers

People subscribed via source and target branches