[Kde-pim] linklocator.cpp change in parsing URLs

Ingo Klöcker kloecker at kde.org
Sun Aug 2 17:26:42 BST 2009


On Sunday 02 August 2009, Martin Koller wrote:
> On Sunday 02 August 2009, Ingo Klöcker wrote:
> > I object. The brackets are excluded so that one can write something
> > like:
> >   Go to the KDE website (www.kde.org).
> >
> > So if you want to allow brackets inside URLs (which I'm perfectly
> > okay with) then please make sure that the above example still
> > works. (I suppose we should add a unit test for this.)
>
> (www.kde.org) still works, the second loop handles this.

Great!

> I have committed the patch and also added a unit test into
> testlinklocator.cpp

Excellent!


Regards,
Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20090802/8ee26057/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list