D28027: fix line rating for new format when function name is missing

[Harald Sitter]

sitter created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
sitter requested review of this revision.

REVISION SUMMARY
  previously this supposedly was
  
  `#2  0x00007fefe9ef79e8 in ?? () at /usr/lib/dri/i965_dri.so`
  
  nowadays the ?? is simply empty. the rating mechanism wasn't properly
  handling this. treat an empty function name to mean the same as ?? and
  rate the line as MissingFunction/Everything depending on whether
  libraryName is also missing.
  
  also update test expectations accordingly (could have actually seen that
  discrepancy if one had paid attention to the outcoming rating :S)
  
  CCBUG: 418538

TEST PLAN
  test passes. most lines from 417435 now rate `MissingFunction` and the whole trace is considered useless
  
    Rating: 128 out of 364 Usefulness: Useless
    90%: 327.6 70%: 254.8 40%: 145.6

REPOSITORY
  R871 DrKonqi

BRANCH
  parse-rate

REVISION DETAIL
  https://phabricator.kde.org/D28027

AFFECTED FILES
  src/parser/backtraceparsergdb.cpp
  src/tests/gdbbacktracelinetest.cpp

To: sitter
Cc: 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200313/7aa4eb29/attachment.html>