<table><tr><td style="">sitter updated this revision to Diff 45938.<br />sitter marked 4 inline comments as done.<br />sitter added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16299">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">check kdnssd version; do not use dnssd discovery if it too old and suffers from signal racing (also see <a href="https://phabricator.kde.org/D16298" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D16298</a>)</li>
<li class="remarkup-list-item">fix typo in function name</li>
<li class="remarkup-list-item">no longer use debug names for udsentry</li>
<li class="remarkup-list-item">convert resolution from fixme to todo, I am unsure if we should even care though. input welcome</li>
<li class="remarkup-list-item">add note about lack of workgroup resolution, it's also unclear if workgroup resolution in its current form has any future. specifically LLMNR (the future replacement of NBS) does not have a concept of SMB workgroups, so the way I see it workgroup discovery is a dead-end anyway. in the future the only way to inspect workgroups would be to actually talk to the SMB host (which may require auth) and as such is impossible to do consistently AFAICT</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R320 KIO Extras</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D16299?vs=43869&id=45938">https://phabricator.kde.org/D16299?vs=43869&id=45938</a></div></div><br /><div><strong>BRANCH</strong><div><div>smb-discovery</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16299">https://phabricator.kde.org/D16299</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
config-runtime.h.cmake<br />
smb/CMakeLists.txt<br />
smb/kio_smb.h<br />
smb/kio_smb_browse.cpp</div></div></div><br /><div><strong>To: </strong>sitter, Frameworks, Dolphin<br /><strong>Cc: </strong>alexde, bcooksley, ngraham, kde-frameworks-devel, kfm-devel, sourabhboss, feverfew, michaelh, spoorun, navarromorales, firef, andrebarros, bruns, emmanuelp, mikesomov<br /></div>