Hi... I would like to commit the patch below. It adds support for vnc URLs as send by krfb and supported by krdc (and keystone). Did I forget anything (documentation?)? Ok to commit? Index: kmmainwin.cpp =================================================================== RCS file: /home/kde/kdenetwork/kmail/kmmainwin.cpp,v retrieving revision 1.474 diff -u -p -r1.474 kmmainwin.cpp --- kmmainwin.cpp 2002/06/12 11:11:33 1.474 +++ kmmainwin.cpp 2002/06/16 18:26:16 @@ -2093,7 +2093,7 @@ void KMMainWin::slotUrlClicked(const KUR } else if ((aUrl.protocol() == "http") || (aUrl.protocol() == "https") || (aUrl.protocol() == "ftp") || (aUrl.protocol() == "file") || - (aUrl.protocol() == "help")) + (aUrl.protocol() == "help") || (aUrl.protocol() == "vnc")) { statusMsg(i18n("Opening URL...")); KMimeType::Ptr mime = KMimeType::findByURL( aUrl ); Index: linklocator.cpp =================================================================== RCS file: /home/kde/kdenetwork/kmail/linklocator.cpp,v retrieving revision 1.1 diff -u -p -r1.1 linklocator.cpp --- linklocator.cpp 2002/04/22 01:33:44 1.1 +++ linklocator.cpp 2002/06/16 18:26:16 @@ -90,6 +90,7 @@ bool LinkLocator::atUrl() const return (ch=='h' && mText.mid(mPos, 7) == "http://") || (ch=='h' && mText.mid(mPos, 8) == "https://") || (ch=='f' && mText.mid(mPos, 6) == "ftp://") || + (ch=='v' && mText.mid(mPos, 6) == "vnc://") || (ch=='m' && mText.mid(mPos, 7) == "mailto:") || (ch=='w' && mText.mid(mPos, 4) == "www.") || (ch=='f' && mText.mid(mPos, 4) == "ftp."); @@ -102,6 +103,7 @@ bool LinkLocator::isEmptyUrl(const QStri url == "http://" || url == "https://" || url == "ftp://" || + url == "vnc://" || url == "mailto" || url == "www" || url == "ftp"; bye... _______________________________________________ KMail Developers mailing list kmail@mail.kde.org http://mail.kde.org/mailman/listinfo/kmail