Attached are two patches. Both are physically large but very simple.

patch_insets
==========
In patch_insets, I modified the Inset::Clone method, so that it is passed a 
Buffer const & parameter. I also modified all Inset c-tors that are passed a 
Buffer *. They are now passed a Buffer const & also (it is impossible tfor an 
inset to be created without a buffer, so this is ok).

Net result is to fix a bug with those insets that store a Buffer * and pass 
this on to a Cloned inset.

I did not modify LyXParagraph to store a Buffer *. This patch is way big 
enough already! Consequently, the Inset::Clone() methods are passed a 
current_view->buffer(). Don't worry; this will change.

With this patch, current_view is eliminated almost entirely from the 
subdirectories. It exists only in a couple of inset classes that are slated 
either for removal or for the GUI-I treatment.

patch_xforms
==========
the updateBufferDependent signal is now connected to 
FormBase::updateOrHide(). If the daughter class should ALWAYS be updated when 
the buffer changes, this is flagged by an enum passed to the FormBase c-tor. 
If it should hide when the buffer changes, then the enum reflects this.

I consider both patches to be bug fixes, so would like them applied before 
1.1.6.

Angus

patch_insets.bz2

patch_xforms.bz2

Reply via email to