D23457: Port regex search to QRegularExpression
Christoph Cullmann
noreply at phabricator.kde.org
Sun Aug 25 22:55:49 BST 2019
cullmann added a comment.
In D23457#519206 <https://phabricator.kde.org/D23457#519206>, @ahmadsamir wrote:
> In D23457#519205 <https://phabricator.kde.org/D23457#519205>, @cullmann wrote:
>
> > Hi, without any further look at the code changes, I don't think an behavior change like "\s can match a newline" is a good idea.
> > Or do I misunderstand that?
>
>
> It's exactly what it says, \s can match a new line char in pcre.
>
> Why do you think this is a bad idea?
Because before it didn't do that in KTextEditor, or?
That means all people that got used to the current behavior will see this as an regression, if it is not optional.
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/20190825/eaa29f8c/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list