Embedding feature slows down updateLabels() very significantly!

2007-09-11 Thread Abdelrazak Younes
I think... Abdel.

Re: Embedding feature slows down updateLabels() very significantly!

2007-09-11 Thread Bo Peng
On 9/11/07, Abdelrazak Younes [EMAIL PROTECTED] wrote: I think... I have not done any profiling but this is possible. Because EmbeddedFiles now saves inset pointers (not ParConstIterator), EmbeddedFiles::update() should be moved out of updateLabels() and iscalled with inset addition and removal.

Re: Embedding feature slows down updateLabels() very significantly!

2007-09-11 Thread Bo Peng
I am not quite sure what is the best way to do this though. With a patch that I just submitted for review, this is the last major problem that needs to be resolved. Basically, I need to call EmbeddedFiles::update() (or emit embeddingChanged() signal) when an inset that has embedded files is

Embedding feature slows down updateLabels() very significantly!

2007-09-11 Thread Abdelrazak Younes
I think... Abdel.

Re: Embedding feature slows down updateLabels() very significantly!

2007-09-11 Thread Bo Peng
On 9/11/07, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: > I think... I have not done any profiling but this is possible. Because EmbeddedFiles now saves inset pointers (not ParConstIterator), EmbeddedFiles::update() should be moved out of updateLabels() and iscalled with inset addition and

Re: Embedding feature slows down updateLabels() very significantly!

2007-09-11 Thread Bo Peng
> I am not quite sure what is > the best way to do this though. With a patch that I just submitted for review, this is the last major problem that needs to be resolved. Basically, I need to call EmbeddedFiles::update() (or emit embeddingChanged() signal) when an inset that has embedded files is