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

Martin Koller kollix at aon.at
Sun Aug 2 13:43:13 BST 2009


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.

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

-- 
Best regards/Schöne Grüße

Martin    ()  ascii ribbon campaign - against html mail 
          /\                        - against microsoft attachments

Geschenkideen, Accessoires, Seifen, Kulinarisches: www.bibibest.at
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20090802/b77a0b26/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