On Thursday 07 December 2000 14:13, Jean-Marc Lasgouttes wrote:
> >>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:
>
> Angus> Attached is a small patch that fixes the crash reported by
> Angus> Martin Vermeer, caused by using current_view before it had been
> Angus> initialised. Not the best way to do this, but much easier than
> Angus> changing Inset::Clone(Buffer const &) to Inset::Clone().
>
> Could you prepare a version for 1.1.5 too?
>
> JMarc
No need. The problem was introduced when I changed Inset::Clone() to
Inset::Clone(Buffer const &). This doesn't exist for 1.1.5 (which loads the
test code fine).
The two line patch I submitted for 1.1.6 will do for now but I've been
convinced by Lars' and Jürgen's arguments that the Insets and LyXParagraph
should not store/know about the buffer. They should be "buffer-transparent"
to quote Lars.
After 1.1.6 is out, I'll submit the REAL patch that reverts to
Inset::Clone(), doesn't store a (Buffer *) in ANY inset and passes a
(Buffer const &) to any Inset method that needs this info. Needless to say,
this patch is a LOT bigger and I'm unwilling to commit it now 'cos it'll
probably break something else.
Angus
