Hey, Initally, I was trying to make red_drawable_unref() not take a DisplayChannel* argument so that it's easier to use from code where you have a RedDrawable but not a DisplayChannel * handy. While doing this, I realized that if we use QXLReleaseInfoExt instead of QXLReleaseInfo in red-parse-qxl.h, then we can simplify more code by no longer passing around the 'group_id' parameter which appears in some function prototypes. This patch series achieves these 2 goals. More cleanups can be done on top of it, for example red_drawable_unref()/red_drawable_new() should be movable to red-parse-qxl.c now. Also, most structures from red-parse-qxl.h also need a qxl->st->qif->release_resource() call when they are no longer used, so a similar treatment to what is done for RedDrawable could be applied to them. Christophe _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel