D5995: Fix CompilerFilterStrategy::errorInLine() to handle paths with whitespace

Morten Volden noreply at phabricator.kde.org
Sun May 28 12:22:46 UTC 2017


volden added a comment.


  In https://phabricator.kde.org/D5995#112230, @kossebau wrote:
  
  > `^([^:\\t]+):` with my little regexp knowledge confuses me a little. Why is the `:` also in the `[^]`? Was that needed for a non-greedy match, at least with the old QRegExp?
  
  
  I want to say they are there for a good reason, but I honestly don't remember it. Maybe they were there from the beginning? IIRC the regexes were originally inspired by some in EMACS.

REPOSITORY
  R33 KDevPlatform

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

To: kossebau, #kdevelop, volden
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170528/062e2f26/attachment.html>


More information about the KDevelop-devel mailing list