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