D28978: [PluginLoader] Replace one usage of QRegExp with QString::startsWith()

Ahmad Samir noreply at phabricator.kde.org
Mon Apr 20 18:30:25 BST 2020


ahmadsamir abandoned this revision.
ahmadsamir added a comment.


  Talking with kbroulik on irc confirms your POV, the format of those patterns aren't documented anywhere so changing the matching now would change the behaviour...
  
  I'll abandon this, (I suggest that X-Plasma-DropUrlPatterns should be documented somewhere so that there's a defined format on which to base the contract of not breaking API :/).

REPOSITORY
  R242 Plasma Framework (Library)

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

To: ahmadsamir, #plasma, apol, broulik
Cc: kde-frameworks-devel, plasma-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200420/15f52f39/attachment.html>


More information about the Kde-frameworks-devel mailing list