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

[Martin Koller]

Hi,

in wanting to fix https://bugs.kde.org/show_bug.cgi?id=201900
which mainly is about allowing "()" chars in an URL, I'd like to propose this patch.
Does anyone know why there is this restriction with "<>()[]" implemented at all ?
Can't this be simply removed ?

 (The comment above:
  // handle cases like this: <link>http://foobar.org/</link>
 only gives a small hint, but still () [] could be removed)


Index: linklocator.cpp
===================================================================
--- linklocator.cpp     (Revision 1005193)
+++ linklocator.cpp     (Arbeitskopie)
@@ -111,7 +111,7 @@
     int start = mPos;
     while ( mPos < (int)mText.length() &&
             mText[mPos] > ' ' && mText[mPos] != '"' &&
-            QString( "<>()[]" ).indexOf( mText[mPos] ) == -1 ) {
+            QString( "<>[]" ).indexOf( mText[mPos] ) == -1 ) {
       ++mPos;
     }

-- 
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/6891874b/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/