Re: [PATCH] Re: BufferStoraghe

2002-10-25 Thread John Levon
On Fri, Oct 25, 2002 at 09:45:40AM +0200, Lars Gullik Bjønnes wrote: > Why the hurry? No hurry. > This code has been there for 5-6 years, why the hurry to rip it out? Quite happy to leave it for a time, just you said you'd come up with a justification for it being there ;) john -- "This is p

Re: [PATCH] Re: BufferStoraghe

2002-10-25 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Tue, Oct 22, 2002 at 03:43:21AM +0100, John Levon wrote: | | > > I do not understand the point in this separate class. | > | > Here's a patch | | Well ? ... Why the hurry? This code has been there for 5-6 years, why the hurry to rip it out? --

Re: [PATCH] Re: BufferStoraghe

2002-10-22 Thread John Levon
On Tue, Oct 22, 2002 at 11:54:15AM +0200, Lars Gullik Bjønnes wrote: > I need to look more closely at this one. The hiding of the > datastructure used for BufferStorage was done on purpose, and I do not > want to just remove it because it is possible... Feel free to report back when you've found

Re: [PATCH] Re: BufferStoraghe

2002-10-22 Thread Lars Gullik Bjønnes
[EMAIL PROTECTED] (Lars Gullik Bjønnes) writes: | John Levon <[EMAIL PROTECTED]> writes: > | | On Tue, Oct 22, 2002 at 03:19:56AM +0100, John Levon wrote: | | | | > I do not understand the point in this separate class. | | | | Here's a patch > | no I need to look more closely at this one. The h

Re: [PATCH] Re: BufferStoraghe

2002-10-21 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Tue, Oct 22, 2002 at 03:19:56AM +0100, John Levon wrote: | | > I do not understand the point in this separate class. | | Here's a patch no -- Lgb

Re: [PATCH] Re: BufferStoraghe

2002-10-21 Thread John Levon
On Tue, Oct 22, 2002 at 03:43:21AM +0100, John Levon wrote: > On Tue, Oct 22, 2002 at 03:19:56AM +0100, John Levon wrote: > > > I do not understand the point in this separate class. > > Here's a patch Additionally, the insetUnlock stuff doesn't seem to be necessary. Juergen !! john

[PATCH] Re: BufferStoraghe

2002-10-21 Thread John Levon
On Tue, Oct 22, 2002 at 03:19:56AM +0100, John Levon wrote: > I do not understand the point in this separate class. Here's a patch john Index: BufferView_pimpl.C === RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/BufferView_pimpl.C