These are the patches that haven't been merged yet. The first patch is fairly straightforward and has only a couple minor changes from the previous review. The second patch has some improvements as well, even though nobody really reviewed it last time. It still contains a lot of open questions (usually marked with 'XXX'). I spent quite a while trying to resolve more of these and didn't feel I was gaining much more insight. I'm not sure it's beneficial for me to spend a lot more time trying to decipher all of this code. In my opinion having some partial documentation with open questions is better than having no documentation, though. Jonathon Jongsma (2): DisplayChannel: start documenting drawable tree DisplayChannel: document exclude_region() functions server/display-channel.c | 254 +++++++++++++++++++++++++++++++++++++++++++++++ server/tree.c | 12 ++- server/tree.h | 6 ++ 3 files changed, 271 insertions(+), 1 deletion(-) -- 2.9.3 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel