kfunk added inline comments.

INLINE COMMENTS

> kateglobal.cpp:104
> +    // disable the QML JIT compiler as a protection against an unknown bug
> +    // in Qt's V4 engine which can provoke a crash in certain of our scripts.
> +    qputenv("QV4_FORCE_INTERPRETER", QByteArrayLiteral("1"));

Please add a reference to the bug reports (Qt JIRA, KDE Bugzilla) here.

> kateglobal.cpp:106
> +    qputenv("QV4_FORCE_INTERPRETER", QByteArrayLiteral("1"));
> +    qCDebug(LOG_KTE) << "QV4_FORCE_INTERPRETER set to" << 
> qgetenv("QV4_FORCE_INTERPRETER");
> +#endif

Minor: "... set to 1" would be sufficient. You're setting it yourself right 
before, so you can presume it is set (=> no need for `qgetenv(...)`.

REPOSITORY
  R39 KTextEditor

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

To: rjvbb, #frameworks
Cc: kfunk, dhaumann, cullmann, kde-frameworks-devel, kevinapavew, demsking, 
head7, sars

Reply via email to