[Differential] [Requested Changes To] D2735: Process QML runtime errors

kfunk (Kevin Funk) noreply at phabricator.kde.org
Sun Sep 11 00:39:30 UTC 2016


kfunk requested changes to this revision.
kfunk added a reviewer: kfunk.
kfunk added a comment.
This revision now requires changes to proceed.


  Sorry, I'm not sure I agree to Sven's request.
  
  Now with this approach, any line with a file path + line no. is considered an error in the output view and will be marked red. There can be false positives.
  
  I'd manually add the possible QML error substrings as before + maybe catch sth. like `path:line: \w+Error:.*` (which matches any error, such as TypeError, ReferenceError, etc.)

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

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

To: cordlandwehr, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160911/db002709/attachment-0001.html>


More information about the KDevelop-devel mailing list