D1231: Add Remote Access interface to KWayland

Roman Gilg noreply at phabricator.kde.org
Sun Mar 25 17:04:55 UTC 2018


romangg accepted this revision.
romangg added a comment.


  Looks fine to me. Tested compilation as well as runtime with KWin's DRM+EGL backend.
  
  - Please change in comments the frameworks version number these changes are landing in.
  - I would like to have @davidedmundson look over this patch a last time. If he doesn't have time until mid-week to do this, you can push.
  - Probably you know this, but please push as one commit only.

INLINE COMMENTS

> remote_access.cpp:139
> +    RemoteBuffer *q;
> +    
> +    qint32 fd = 0;

rm whitespace

> remote_access_interface.cpp:118
> +};
> +    
> +class RemoteAccessManagerInterface::Private : public Global::Private

rm whitespace

> remote_access_interface.h:38
> + *     (stored in manager's sent list)
> + *  2. Clients confirm that they wants this buffer, the RemoteBuffer 
> + *     interfaces are then created and wrapped around BufferHandle.

rm whitespace (at the end)

> remote_access_interface.h:42
> + *     RemoteBuffer notifies manager and release signal is emitted.
> + * 
> + *     It's the responsibility of your process to delete this BufferHandle

rm whitespace

> remote_access_interface.h:55
> +    void setFormat(quint32 format);
> +  
> +    qint32 fd() const;

rm whitespace

> remote_access_interface_p.h:29
> +{
> +    
> +/**

rm whitespace

REPOSITORY
  R127 KWayland

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

To: Kanedias, graesslin, davidedmundson, romangg
Cc: jtamate, jgrulich, romangg, ngraham, alexeymin, #frameworks, davidedmundson, plasma-devel, ragreen, schernikov, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180325/79312e87/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list