D23915: Improve documentationSome of the documentation did not say what the purpose of therespective class was, but rather gave a tautology.For people new to the code base it helps to mention thatDPMS is power management.That DataDevice is for copy...

Frederik Gladhorn Sat, 14 Sep 2019 02:08:07 -0700

gladhorn added inline comments.

INLINE COMMENTS


> romangg wrote in dpms.h:44
> Yea, maybe just write that this class is meant to create the DPMS object 
> below only. You already have the information you provided here there then.

I agree. The first two lines are actually repetitions and they don't say what 
the class is good for at all. I'll update this.

REPOSITORY
  R127 KWayland

BRANCH
  arcpatch-D23915

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

To: gladhorn, #kwin, romangg
Cc: romangg, zzag, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D23915: Improve documentationSome of the documentation did not say what the purpose of therespective class was, but rather gave a tautology.For people new to the code base it helps to mention thatDPMS is power management.That DataDevice is for copy...

Reply via email to