Review Request: Convert file content to local encoding before uploading text file in the "share" dataengine

Commit Hook null at kde.org
Mon Oct 29 00:18:55 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107044/#review21076
-----------------------------------------------------------


This review has been submitted with commit cb3cf3fbae286d56aa161c351dfc794f16bfbcef by Yichao Yu to branch master.

- Commit Hook


On Oct. 25, 2012, 8:35 p.m., Yichao Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107044/
> -----------------------------------------------------------
> 
> (Updated Oct. 25, 2012, 8:35 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> When dragging a text file with utf8 characters to the PasteBin plasma applet, it will upload the file in the wrong encoding. This patch decode the file content using current locale which should be a better option than the default one (Latin1?).
> 
> 
> Diffs
> -----
> 
>   plasma/generic/dataengines/share/shareprovider.cpp 81ac677 
> 
> Diff: http://git.reviewboard.kde.org/r/107044/diff/
> 
> 
> Testing
> -------
> 
> Compiled, installed, upload a file with utf8 characters and the encoding is correct.
> 
> 
> Thanks,
> 
> Yichao Yu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121029/df028499/attachment.html>


More information about the Plasma-devel mailing list