<table><tr><td style="">bruns marked an inline comment as done.<br />bruns added inline comments.
</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/D16266">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16266#inline-88876">View Inline</a><span style="color: #4b4d51; font-weight: bold;">poboiko</span> wrote in <span style="color: #4b4d51; font-weight: bold;">filecontentindexer.cpp:91</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">OK, we can simply count how many times we got <tt style="background: #ebebeb; font-size: 13px;">finishedIndexingFile</tt>, and just go to the corresponding position in the batch.<br />
It's just the binary search here does look a bit unnecessary to me...</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Counting would give a good hint which one failed, but still no guarantee. Files may be added or removed during the indexer run, so the position is only approximate. The indexer may also have sent a "finished" message, and crash afterwards in a destructor call.</p>

<p style="padding: 0; margin: 8px;">Doing a binary search is straight forward and avoids any dependencies or assumptions about other parts of the code.</p>

<p style="padding: 0; margin: 8px;">Also, this code should be only temporary anyway - if the extractor is run in a separate process which only receives the file using a readonly file descriptor (for sandboxing) and passes back the result, the problematic documents id is known by the parent process.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16266">https://phabricator.kde.org/D16266</a></div></div><br /><div><strong>To: </strong>bruns, Baloo, Frameworks, poboiko, ngraham<br /><strong>Cc: </strong>broulik, apol, kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>