Re: r18435 - /lyx-devel/branches/personal/sts/dynmacro/src/ma...

2007-05-25 Thread Lars Gullik Bjønnes
Stefan Schimanski <[EMAIL PROTECTED]> writes: | > | auto_ptr MathMacro::doClone() const | > | { | > | - return auto_ptr(new MathMacro(*this)); | > | + MathMacro * copy = new MathMacro(*this); | > | + copy->needsUpdate_ = true; | > | + copy->expanded_.cell(0).clear(); | > | + return auto_ptr(copy

Re: r18435 - /lyx-devel/branches/personal/sts/dynmacro/src/ma...

2007-05-25 Thread Stefan Schimanski
| auto_ptr MathMacro::doClone() const | { | - return auto_ptr(new MathMacro(*this)); | + MathMacro * copy = new MathMacro(*this); | + copy->needsUpdate_ = true; | + copy->expanded_.cell(0).clear(); | + return auto_ptr(copy); | } Why not: auto_ptr tmpInset(new MathM

Re: r18435 - /lyx-devel/branches/personal/sts/dynmacro/src/ma...

2007-05-24 Thread Lars Gullik Bjønnes
[EMAIL PROTECTED] writes: | Author: sts | Date: Mon May 21 08:27:32 2007 | New Revision: 18435 | | URL: http://www.lyx.org/trac/changeset/18435 | Log: | * cloned macros need an update because the argument proxies point to the old macro | | Modified: | lyx-devel/branches/personal/sts/dynmacr