Hi,
On Fri, Dec 31, 2010 at 07:30:59PM +0200, Pauli wrote:
> - if (handlerDeleted)
> + if (block.deleted)
> {
> - for (i = 0; i < numHandlers;)
> - if (handlers[i].deleted)
> - {
> - for (j = i; j < numHandlers - 1; j++)
> - handlers[j] = handlers[j+1];
> - numHandlers--;
> - }
> - else
> - i++;
> - handlerDeleted = FALSE;
> + for (i = 0; !block.deleted[i]; i++) {
> + }
> +
> + for (j = i + 1; j < block.num; j++) {
> + if (block.deleted[j])
> + continue;
> + block.handlers[i] = block.handlers[j];
> + i++;
> + }
> + block.num -= j - i;block.num = i + 1 would be a fair bit clearer here. But aside from that, nice! For the series: Reviewed-by: Daniel Stone <[email protected]> Cheers, Daniel
signature.asc
Description: Digital signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
