D22211: [platforms/drm] Refactor out Edid class

Vlad Zagorodniy noreply at phabricator.kde.org
Tue Jul 2 16:02:09 BST 2019


zzag added inline comments.

INLINE COMMENTS

> davidedmundson wrote in drm_output.cpp:382
> If we hit this return with this patch m_edid changes from being merely invalid to being null. Something we don't guard for.

Yeah... Are there actually drivers that do not expose EDID property?

Just to be sure I'll add default constructor to EDID class.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: davidedmundson, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, 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/20190702/2d5d3e80/attachment.html>


More information about the kwin mailing list