[Differential] [Commented On] D3203: icalformat: remove QByteArray->QString->QByteArray roundtrip.

[knauss (Sandro Knauß)]

knauss added a comment.


  Okay I was wrong - it is other way round.
  
  QFile.readAll() is using QTextCodec::codecForLocale() for reading the file, that means, that if the user has a strange locale the file is read with this locale.
  If we want to define the encoding we need to use QTextStream ( see Qt documentation for QFile ).

REPOSITORY
  rKCALENDARCORE KCalendar Core

REVISION DETAIL
  https://phabricator.kde.org/D3203

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: dfaure, djarvie, smartins
Cc: knauss, kde-pim, #kde_pim, spencerb, dvasin, winterz, vkrause, mlaurent, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20161031/8f4c0ed6/attachment.html>