<table><tr><td style="">sitter created this revision.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />sitter requested review of this revision.
</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/D28027">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>previously this supposedly was</p>

<p><tt style="background: #ebebeb; font-size: 13px;">#2  0x00007fefe9ef79e8 in ?? () at /usr/lib/dri/i965_dri.so</tt></p>

<p>nowadays the ?? is simply empty. the rating mechanism wasn't properly<br />
handling this. treat an empty function name to mean the same as ?? and<br />
rate the line as MissingFunction/Everything depending on whether<br />
libraryName is also missing.</p>

<p>also update test expectations accordingly (could have actually seen that<br />
discrepancy if one had paid attention to the outcoming rating :S)</p>

<p>CCBUG: 418538</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>test passes. most lines from 417435 now rate <tt style="background: #ebebeb; font-size: 13px;">MissingFunction</tt> and the whole trace is considered useless</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">Rating: 128 out of 364 Usefulness: Useless
90%: 327.6 70%: 254.8 40%: 145.6</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R871 DrKonqi</div></div></div><br /><div><strong>BRANCH</strong><div><div>parse-rate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28027">https://phabricator.kde.org/D28027</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/parser/backtraceparsergdb.cpp<br />
src/tests/gdbbacktracelinetest.cpp</div></div></div><br /><div><strong>To: </strong>sitter<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>