D28882: Create protocol to manage video feeds

Méven Car noreply at phabricator.kde.org
Mon Apr 27 14:04:44 BST 2020


meven added inline comments.

INLINE COMMENTS

> screencasting.cpp:141
> +    }
> +    void zkde_screencast_unstable_v1_removeSource(uint32_t sourceId) override {
> +        auto it = std::find_if(m_sources.begin(), m_sources.end(), [sourceId] (const ScreencastingSource &source) {

Missing blank line

> screencasting.cpp:156
> +
> +
> +

Two blank lines too mang

REPOSITORY
  R127 KWayland

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

To: apol, #kwin, jgrulich, davidedmundson
Cc: meven, davidedmundson, romangg, zzag, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200427/49a7da69/attachment.html>


More information about the Kde-frameworks-devel mailing list