[Spice-devel] [PATCH v3 0/4] Unify Glib event loop

Frediano Ziglio fziglio at redhat.com
Wed Jan 13 03:39:05 PST 2016


This patchset unify event loop code between test and patch to use
Glib in main loop.
Although we don't have a precise conclusion about the final patch
I think that having common code better tested is better than having
to use the entire spice-server in order to check this part of code.
This version change the way the loop context is retrieved using
newer patches to add a context to SpiceCoreInterface.

First patch change test loop removing the usage of default context
to make sure we process only the events we explicitly add.
Second patch extract code into a separate file.
Third patch add a test for the event code.
Forth is the updated original patch for Glib event code use for
RedWorker. Not (currently) intended to be merged.

Frediano Ziglio (4):
  tests: do not use default loop context
  tests: extract code for event loop
  tests: add a test for event loop
  worker: use glib main loop

 server/Makefile.am              |   4 +-
 server/event-loop.c             | 185 ++++++++++++++++++++++++++++
 server/event-loop.h             |  34 +++++
 server/red-worker.c             | 253 +++++++++++++++----------------------
 server/red-worker.h             |   1 +
 server/spice_timer_queue.c      | 267 ----------------------------------------
 server/spice_timer_queue.h      |  44 -------
 server/tests/Makefile.am        |   8 ++
 server/tests/basic_event_loop.c | 160 ++++++------------------
 server/tests/basic_event_loop.h |   3 +
 server/tests/replay.c           |  18 ++-
 server/tests/test_loop.c        | 134 ++++++++++++++++++++
 12 files changed, 512 insertions(+), 599 deletions(-)
 create mode 100644 server/event-loop.c
 create mode 100644 server/event-loop.h
 delete mode 100644 server/spice_timer_queue.c
 delete mode 100644 server/spice_timer_queue.h
 create mode 100644 server/tests/test_loop.c

-- 
2.4.3



More information about the Spice-devel mailing list