[Differential] [Commented On] D327: Optimize IndexedString::byteArray

[mwolff (Milian Wolff)]

mwolff added a subscriber: mwolff.
mwolff added a comment.

This is very unsafe. When you put QByteArray anywhere and then destroy the IndexedString it may (eventually) trigger a cleanup of the item repo data, which will make the QByteArray reference destroyed data.

Where is this actually required? If possible, I want to delete the bytearray getter completely, as it can lead to unicode issues.


REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

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

To: kfunk, KDevelop
Cc: mwolff, kdevelop-devel, arrowdodger, obogdan, kfunk