>>>>> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:

Abdelrazak> Here is the patch. It took me a bit longer than I thought
Abdelrazak> because I did not understand the logic in the cursor in
Abdelrazak> breakParagraph(). With my former patch one call to
Abdelrazak> needsUpdateCounter on next_par was enough. With the
Abdelrazak> ParIterator version I need to test both current and next
Abdelrazak> par in order to cover all the cases. Anyway, double-check
Abdelrazak> is maybe better.

It is strange that the need for a double call is different.

Abdelrazak> OK to apply to 1.5svn?

Yes for me. I like the patch.

Abdelrazak> Maybe it is too late for 1.4.1 but please give it a test
Abdelrazak> ride. There is nothing intrusive in there.

It is indeed late for 1.4.1, but it will definitely go in 1.4.2.

BTW, do you really need the default: case in your switch? It is better
to avoid them in general...

JMarc

Reply via email to