The feature should now be up and running, although it might have some performance issues. Please test with small changes :S as it scales quadratic with the size of a single change.

If I look at DocRange::length() there is certainly a lot of room for improvement ;-)

Only that it isn't really in the critical path.

Maybe there is a reason why you didn't do it that way?

Well, I copy-pasted it and I probably didn't have a second look at it.

Another reason is that you counted the first paragraph twice, and forgot the +1 ;).. Anyway, I committed it.

Thanks,

Vincent

Reply via email to