<table><tr><td style="">loh.tar 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/D21287">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>the path is appended behind the document name. What about this solution?</p></blockquote>

<p>Don't like that idea. And fit not well to the shown "Document List" problem.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>to be efficient one could hash all documents</p></blockquote>

<p>I'm always in favor for such. But here I had no idea how to do that nicely and reliable. One Document is not the place to hold such hash, and the patch lives only there.</p>

<p>The "file rename case" may indeed not catch here when it goes from duplicate to unique, but how often is that the case? Guess nearby never and it is no big harm to keep the prefix in that case. "Untitled" documnets are handled as before.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>that would avoid that one needs to iterate always again over all documents to collect duplicates.</p></blockquote>

<p>Is a loop running once over 10 to 1000 (one should rethink his workflow when he has so many files open) turns on a file name update such an issue and any optimization worth? Well, currently lead that on session restore to one million loops, how long may take that? And compared to the other needed work to open one file. Guess it's not a big thing.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21287">https://phabricator.kde.org/D21287</a></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor<br /><strong>Cc: </strong>dhaumann, cullmann, kwrite-devel, kde-frameworks-devel, KTextEditor, domson, michaelh, ngraham, bruns, demsking, sars<br /></div>