<table><tr><td style="">markg marked 2 inline comments as done.<br />markg 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/D10414" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I'm not afraid of the compiler generating wrong code. That is the least of my worries :)<br />
What i am afraid of (which is why i suggested the copy test) is a wrong optimization at some point (like adding a pointer member but forgetting to properly copy it) which would break the UDSEntry.<br />
But a test for that would probably deserves its own review independent of this.</p>

<p>Forget it, i have no plans to change the internal data structure of UDSEntry. I might play with a <a href="https://github.com/Tessil/hat-trie" class="remarkup-link" target="_blank" rel="noreferrer">HAT-trie</a> at some point but that is unlikely to be on the UDSEntry level. It would be much more suitable on the KCoreDirLister level.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10414" rel="noreferrer">https://phabricator.kde.org/D10414</a></div></div><br /><div><strong>To: </strong>markg, dfaure<br /><strong>Cc: </strong>apol, Frameworks, michaelh<br /></div>