rgheck wrote:
Abdelrazak Younes wrote:
OK, I see. Well, I think it works that way, but only because
doRecordUndo() only gets called with isFullBuffer being true at this
one point in the code. If it ever got called that way from
elsewhere, you'd have a problem.
I don't see this could happen but we should make sure this doesn't
happen indeed. FWIW, this is the same technique as in mathed.
So it seems to me to be clearer to do this in the constructor.
But then there is no optimisation AFAICS.
Sure there is: You only do the copy if isFullBuffer is true.
OK, but you don't save the BufferParam creation. I've been using LyX
with my patch today without any problem...
Abdel.
