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