D9290: [filewidgets] Fix create path
Anthony Fieroni
noreply at phabricator.kde.org
Wed Dec 13 07:32:56 UTC 2017
anthonyfieroni added inline comments.
INLINE COMMENTS
> pathhelpers_p.h:31
> +{
> + if (!path1.endsWith(QLatin1Char('/'))) {
> + return path1 + QLatin1Char('/') + path2;
To be absolutely sure we can add pedantic check
if (!path1.endsWith('/') && !path2.startsWith('/'))
?
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D9290
To: anthonyfieroni, #frameworks, dfaure, hein, aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171213/5e299193/attachment.html>
More information about the Kde-frameworks-devel
mailing list