Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Player butchers unicode tags #4

Closed
Clementine-Issue-Importer opened this issue Dec 6, 2013 · 7 comments
Closed

Player butchers unicode tags #4

Clementine-Issue-Importer opened this issue Dec 6, 2013 · 7 comments

Comments

@Clementine-Issue-Importer

From thegodlikehobo on February 23, 2010 03:40:46

What steps will reproduce the problem? 1. Set a unicode tag (e.g. ID3, AAC metadata)
2. See set info turn into craziness

For example, when I attempt to set 絢香 as tag for a file, Clementine
writes 絢� What version of the product are you using? On what operating system? Clementine 0.1-1 (via clementine_0.1-1_amd64.deb)
Linux

Original issue: http://code.google.com/p/clementine-player/issues/detail?id=4

@Clementine-Issue-Importer
Copy link
Author

From davidsansome on February 23, 2010 06:57:51

Thanks, I've just fixed this in SVN

Status: Fixed
Owner: davidsansome

@Clementine-Issue-Importer
Copy link
Author

From shuizhuyuanluo on February 23, 2010 07:02:35

A proposed patch.
It should fix this issue.

I have tested it with clementine 0.1.

;)

Attachment: clementine-0.1-fix_utf8_tag_saving.patch

@Clementine-Issue-Importer
Copy link
Author

From davidsansome on February 23, 2010 07:05:32

Thanks for the patch! Looks like I got there just before you though :) http://code.google.com/p/clementine-player/source/diff? spec=svn222&r=222&format=side&path=/trunk/src/song.cpp

@Clementine-Issue-Importer
Copy link
Author

From shuizhuyuanluo on February 23, 2010 07:06:04

wow.
thanks to clementine developer for such a fast fix!

@Clementine-Issue-Importer
Copy link
Author

From thegodlikehobo on February 23, 2010 09:08:29

Thanks for the patch, it works perfectly!

@Clementine-Issue-Importer
Copy link
Author

From davidsansome on February 24, 2010 01:48:56

Issue 15 has been merged into this issue.

@Clementine-Issue-Importer
Copy link
Author

From davidsansome on March 04, 2010 05:00:50

Issue 55 has been merged into this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant