apol added a comment.
LGTM otherwise INLINE COMMENTS > urlhandler.cpp:55 > + // not a KDE application > + QDesktopServices::openUrl(u); > } won't this recurse forever? REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27616 To: ahmadsamir, #frameworks, apol Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
