On Sat, Mar 27, 2004 at 01:16:33PM +0100, Alfredo Braunstein wrote:
> Ok, your testcase works fine with Andre's patch and this implementation of
> forwardInset.


+void DocumentIterator::forwardInset()
+{
+       while (depth()) {
+               size_t old_depth = depth();
+
+               forwardPos();
+
+               if (depth() > old_depth)
+                       break;
+       }
+}

Aehm, forget the last mail. If I am understanding this correctly,
this touches only 'descendable insets', not all insets. This might be
sufficient for some usages but not for 'goto label' or similar.

-       forwardPos(); 
-       while (size() != 0 && pos() == lastpos())
-               forwardPos();
+       do {
+               forwardPos(); 
+       } while (size() != 0 && pos() == lastpos());

I don't like do...while loops. Certainly a matter of taste, though...

Andre'

Reply via email to