[Differential] [Commented On] D4201: Make it possible to lower KCrash to tier 1

[David Faure]

dfaure added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in kcrash.cpp:437
> You have a comment to follow :)

You can't use QVector here. We're crashing, likely due to corrupted memory, the whole point of this code is to NOT do any memory allocation. You should revert to const char*[], or use something that always allocates on the stack (e.g. QVarLengthArray, if preallocated to be big enough).

REPOSITORY
  R285 KCrash

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

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

To: apol, #frameworks, dfaure
Cc: anthonyfieroni, graesslin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170119/1019f3d8/attachment-0001.html>