By the way, it's nice to know which repository the patch applies to by looking at the email subject. It just helps get into the proper frame of mind quickly before starting to review. You can set that using the format.subjectprefix configuration parameter. For example, I use the following configuration for spice- gtk: [format] subjectprefix = PATCH spice-gtk Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx> On Fri, 2016-10-28 at 16:50 +0200, Francois Gouget wrote: > Signed-off-by: Francois Gouget <fgouget@xxxxxxxxxxxxxxx> > --- > src/spiceqxl_io_port.c | 11 ----------- > 1 file changed, 11 deletions(-) > > diff --git a/src/spiceqxl_io_port.c b/src/spiceqxl_io_port.c > index 165b8a9..6721d3f 100644 > --- a/src/spiceqxl_io_port.c > +++ b/src/spiceqxl_io_port.c > @@ -44,17 +44,6 @@ > ret = &m_item- > >el; \ > } > > -#undef SPICE_RING_CONS_ITEM > -#define SPICE_RING_CONS_ITEM(r, ret) > { \ > - typeof(r) start = > r; \ > - typeof(r) end = r + > 1; \ > - uint32_t cons = (r)->cons & > SPICE_RING_INDEX_MASK(r); \ > - typeof(&(r)->items[cons]) m_item = &(r)- > >items[cons]; \ > - if (!((uint8_t*)m_item >= (uint8_t*)(start) && > (uint8_t*)(m_item + 1) <= (uint8_t*)(end))) { \ > - abort(); > \ > - } > \ > - ret = &m_item- > >el; \ > - } > > static int spiceqxl_io_port_debug_level = -1; > _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel