D23457: Port regex search to QRegularExpression
Dominik Haumann
noreply at phabricator.kde.org
Tue Dec 10 21:11:25 GMT 2019
dhaumann added a comment.
In D23457#574846 <https://phabricator.kde.org/D23457#574846>, @ahmadsamir wrote:
> Personally, I am not convinced replacing \s with [ \t], and deviating from PCRE default behaviour is a good idea in this case.
That is not to be discussed, and Christoph tried to explain this already before.
In short: If these issues don't get fixed to be compatible with previous behavior, this patch will not get merged. I'd be very much in favor of having the compatibility, since QRegularExpression is the way forward, but adding regressions is a big no-go and not up for discussion.
We are maintaining Kate for close to 20 years now. We'd like to ask you to give us some trust in our decision.
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D23457
To: ahmadsamir, #ktexteditor, dhaumann, cullmann
Cc: kde-frameworks-devel, kwrite-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191210/7bdfef33/attachment.html>
More information about the Kde-frameworks-devel
mailing list