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

Reply via email to