----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120471/ -----------------------------------------------------------
(Updated Oct. 7, 2014, 1:37 a.m.) Review request for kwin, Plasma and Martin Gräßlin. Changes ------- Add tests for Registry::sync signal This tests two scenarios: - Simply setting a Registry from wl_display, then initializing it, so it emits sync - Using ConnectionThread, EventQueue, and all that in a different QThread. The latter test currently fails due to SocketNotifier apparently not being thread-safe. This also fails in other tests, and it works as soon as I comment thread->moveToThread(ConnectionThread), it is unrelated. Note that in libkscreen, I don't have that issue so far, so it also works in the threaded scenario, in principle. Repository: kwayland Description ------- Add Registry::sync() signal Emitted when the Wayland display is done flushing the initial interface callbacks, announcing wl_display properties. This can be used to compress events. Note that this signal is emitted only after announcing interfaces, such as outputs, but not after receiving callbacks of interface properties, such as the output's geometry, modes, etc.. This signal is emitted from the wl_display_sync callback. For this, we add a wl_callback_listener to the registry's Private, enqueue its events properly, if necessary, and trigger the signal through a callback mechanism similar to the wl_registry callbacks. This signal allows users of the API to find out when the signal emissions, such as outputAnnounced, etc. for all currently existing interfaces is complete. Diffs (updated) ----- autotests/client/test_wayland_connection_thread.cpp a1ef989 src/client/registry.h 9e63a2b src/client/registry.cpp 22f9484 Diff: https://git.reviewboard.kde.org/r/120471/diff/ Testing ------- tests in libkscreen exercise this feature, it works as expected, meaning I can notify when all initial synchronization is done. Thanks, Sebastian Kügler
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
