D21874: Overhaul AbstractOutput

[Vlad Zagorodniy]

zzag added inline comments.

INLINE COMMENTS

> davidedmundson wrote in abstract_output.h:109
> I got mixed up in my explanation, but I think my original comment was right.
> 
> if something is 1/ms  that's once per millisecond. i.e thousand times per seconds
> which makes it 1kHz 
> not 1mHz

> Current refresh rate in 1/ms.

Given this description it should be kHz, but 1/ms is incorrect assumption. The calculated refresh rate is in mHz (millihertz), see https://gitlab.freedesktop.org/wayland/weston/blob/master/libweston/backend-drm/drm.c#L4769-4786.

REPOSITORY
  R108 KWin

BRANCH
  overhaul-abstract-output

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

To: zzag, #kwin, davidedmundson
Cc: broulik, cfeck, davidedmundson, kwin, LeGast00n, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190617/75da3396/attachment.html>