>>>>> "Alfredo" == Alfredo Braunstein <[EMAIL PROTECTED]> writes:

Alfredo> Alfredo Braunstein wrote:
>> The problem has nothing to do with env depths, just with cut/past
>> of empty paragraphs. The problem is that Text::acceptChanges on the
>> internal cut/paste paragraph list calls DEPM and this removes an
>> empty paragraph at the beginning.
>> 
>> The fix is to move the bulk of acceptChanges to paragraph_funcs
>> (where it belongs IMO) and the version in Text just calls that and
>> then DEPM, while in cap we just call the plain version.

Alfredo> ... and the patch.

I like it (fixes a bug, does useful cleanup). You should probably remove the
        // finally, invoke the DEPM
comment which looks a bit strange in such a small method.

OK

JMarc

Reply via email to