Abdelrazak Younes wrote:

> Looks good Alfredo but one question: why don't you just move
> deleteEmptyParagraphMechanism() to paragraph_funcs as well? Same thing
> for Text::fixCursorAfterDelete(), this has nothing to do with Text
> AFAIS...

I see your point. However, I think they don't belong to paragraph_funcs
because both involve Cursors while the other functions in paragraph_funcs
are more elementary. 

In any case this is orthogonal to the patch, as I don't call DEPM directly
(neither before nor after the patch). I propose to leave this
reorganization for after 1.5.0.

A/


Reply via email to