D29247: [effects/showfps] Calculate "correct" FPS

Vlad Zahorodnii <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
zzag created this revision.
zzag added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
zzag requested review of this revision.

REVISION SUMMARY
  cca0e15b455ba6 <https://phabricator.kde.org/R108:cca0e15b455ba606043c720479490794d8d86309> broke code that calculates the value of the fps counter.
  
  The frames field stores timestamps of previous frames, not time between
  each two consecutive frames.
  
  This change doesn't attempt to make the show fps compute more accurate
  performance metrics, e.g. how much times it takes to execute rendering
  commands on the GPU, etc.

TEST PLAN
  The Show FPS effect displays more sensible values.

REPOSITORY
  R108 KWin

BRANCH
  showfps-correct-timestamp

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

AFFECTED FILES
  effects/showfps/showfps.cpp
  effects/showfps/showfps.h

To: zzag, #kwin
Cc: kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.