On Monday 04 December 2000 18:45, Lars Gullik Bjønnes wrote:
> Dekel Tsur <[EMAIL PROTECTED]> writes:
> | On Mon, Dec 04, 2000 at 03:56:53PM +0100, Lars Gullik Bjønnes wrote:
> | > Angus Leeming <[EMAIL PROTECTED]> writes:
> | > | This is dying in Buffer::isLatex()
> | >
> | > Are you sure that inlining is not playing tricks on you?
> | >
> | > Seems almost liek isLatex is beeing called on a buffer that has just
> | > been deleted...
> |
> | The real problem is the following line in
> | LyXParagraph::CopyIntoMinibuffer(): minibuffer_inset =
> | GetInset(pos)->Clone(*current_view->buffer()); As current_view hasn't
> | been initialized yet, *current_view->buffer()) is junk. I've already
> | mentioned this problem.
> | Possible solution are
> | 1) Change the first parameter of CopyIntoMinibuffer() from
> | 'BufferParams const &' to 'Buffer const &'.
> | 2) Do not call to CopyIntoMinibuffer in LyXTabular::OldFormatRead().
>
> I would actually prefere both...
>
> Lgb
Actually, I think that the "correct:" fix is to remove the (Buffer const &)
from Inset::Clone() and not to store Buffer * in any Inset. That way, we can
continue to pass a BufferParams to LyXParagraph::CopyIntoMinibuffer() and
current_view dissapears from paragraph.C.
Angus
