D9160: QList -> QVector where sizeof(type) > sizeof(T*)

Milian Wolff noreply at phabricator.kde.org
Mon Dec 4 09:14:35 UTC 2017


mwolff added a comment.


  In https://phabricator.kde.org/D9160#175540, @brauch wrote:
  
  > Hmm, this will break build of all the plugins, no? Other than that, I'm in favour of this change, thanks for the work!
  >
  > Only thing I'm sceptical about is QStringList, esp. in tests or so it can have useful  API over QVector<QString> ... but ok, if none of that is needed, fine ;)
  
  
  `QString` has `sizeof(void*)`, this change should not touch it. Why did you do that Friedrich?

REPOSITORY
  R32 KDevelop

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

To: kossebau, #kdevelop
Cc: mwolff, brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171204/462b5356/attachment.html>


More information about the KDevelop-devel mailing list