kossebau added inline comments. INLINE COMMENTS
> kio_smb_browse.cpp:239 > + QString host = m_current_url.host(); > + host = host.remove(host.size() - wsdSuffix.size(), wsdSuffix.size()); > + const QString dnssd(host + ".local"); `QString::remove()` operates on the object itself, no need to assign back to host. Besides, why not use `QString::chop(wsdSuffix.size())` ? REPOSITORY R320 KIO Extras BRANCH smb-lazy-resolve REVISION DETAIL https://phabricator.kde.org/D28513 To: sitter, ngraham, meven Cc: kossebau, kde-frameworks-devel, kfm-devel, nikolaik, pberestov, iasensio, fprice, LeGast00n, cblack, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov