romangg added inline comments.

INLINE COMMENTS

> romangg wrote in xdgoutput_interface.cpp:83
> This variable is only relevant until the compositor has sent for the first 
> time the done event via `XdgOutputInterface::done()`. Afterwards it is always 
> true. Is it meant this way? If yes, could we use another name making this 
> fact more explicit, like: `doneOnce`?
> 
> On the other side what if a compositor never sends the done event via 
> `XdgOutputInterface::done()` because the xdg output never changes its logical 
> size or position after the compositor publishes the xdg output to clients and 
> it expects clients to receive the current values on connecting to the 
> resource?

The inline comment is in the wrong line. It is referring to the `done` member 
variable.

REPOSITORY
  R127 KWayland

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

To: davidedmundson, romangg
Cc: #frameworks, michaelh, bruns

Reply via email to