It looks like renaming miSubtractSpans to SubtractSpans was an accident? With that fixed, this looks perfectly sensible. Reviewed-by: Jamey Sharp <jamey at minilop.net> Jamey