Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-08 Thread Andre Poenitz
On Fri, Jul 08, 2005 at 11:13:36AM +0200, Georg Baum wrote: > This is now in. A working undo with cursor jump is better than a nonworking > undo. We can still change that if anybody has a better idea. Sensible choice. Andre'

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-08 Thread Andre Poenitz
On Wed, Jul 06, 2005 at 10:26:49AM +0200, Georg Baum wrote: > I am not sure what you meant exactly, but I exchanged recordUndo for > recordUndoInset in some places. Even better than what I had in mind. I forgot about recordUndoInset. > The problem is that recordUndo only records changes to the cu

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-08 Thread Georg Baum
Georg Baum wrote: > I am not sure what you meant exactly, but I exchanged recordUndo for > recordUndoInset in some places. The problem is that recordUndo only > records changes to the current cell, and that is wrong if we e. g. add a > column. The attached patch fixes the undo problem in math grid

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-06 Thread Georg Baum
Andre Poenitz wrote: > On Sun, Jul 03, 2005 at 11:03:23AM +0200, Georg Baum wrote: >> BTW André, do you have any idea why undoing an added column in mathed >> does not work? > > If it does (and I can't check right now) possibly because of this > special idx fixing. > > An in-scheme-workaround fo

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-06 Thread Georg Baum
Andre Poenitz wrote: > On Sun, Jul 03, 2005 at 11:03:23AM +0200, Georg Baum wrote: >> The attached patch adds some documentation, removes col_ and row_ >> again, fixes wrong index calculation for "delete-column" and even >> fixes one further case of cursor misadjustment in insettabular. I am >> g

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-04 Thread Andre Poenitz
On Sun, Jul 03, 2005 at 11:03:23AM +0200, Georg Baum wrote: > Am Samstag, 2. Juli 2005 20:43 schrieb Andre Poenitz: > > > For grid-like structures it is sometimes convienient to extract row > > and col from idx. So there are some helper functions. And sometimes > > it is useful to give the user th

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-04 Thread Andre Poenitz
On Sat, Jul 02, 2005 at 11:23:44PM +0300, Martin Vermeer wrote: > While most of what you write has merit, I respectfully want to play the > devil's advocate with the attached patch, which just bites the bullet > and re-writes all the idx()-as-lvalue statements. It eliminates idx_ and > makes row_,

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-03 Thread Georg Baum
Am Samstag, 2. Juli 2005 22:23 schrieb Martin Vermeer: > But it is essentially mostly a dumb substitution exercise. What I want > to argue is that this is a _fundamental_ fix for the need for (and > sometimes wrong/missing) adjustments, not a kludge. When I wrote my earlier mail I did not read th

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-03 Thread Georg Baum
Am Samstag, 2. Juli 2005 20:43 schrieb Andre Poenitz: > For grid-like structures it is sometimes convienient to extract row and > col from idx. So there are some helper functions. And sometimes it is > useful to give the user the impression that row and col are primary > because that's what the us

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-02 Thread Martin Vermeer
On Sat, Jul 02, 2005 at 08:43:22PM +0200, Andre Poenitz wrote: > On Fri, Jul 01, 2005 at 11:43:45PM +0300, Martin Vermeer wrote: > > OK, I see it now: idx_type should be a struct with two int members, .row > > and .col. And suitable operator definitions added. No duplication of > > information, and

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-02 Thread Andre Poenitz
On Fri, Jul 01, 2005 at 11:43:45PM +0300, Martin Vermeer wrote: > OK, I see it now: idx_type should be a struct with two int members, .row > and .col. And suitable operator definitions added. No duplication of > information, and still "idx() = ..." will work. Does that sound like a > plan? I stil

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-01 Thread Martin Vermeer
On Fri, Jul 01, 2005 at 11:21:12PM +0300, Martin Vermeer wrote: > On Fri, Jul 01, 2005 at 09:23:59PM +0200, Andre Poenitz wrote: ... > > I don't agree with the idea of storing row and column in the cursor. > > This duplicates information and will lead to synchronization troubles. > > Actually I

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-01 Thread Martin Vermeer
On Fri, Jul 01, 2005 at 09:23:59PM +0200, Andre Poenitz wrote: > On Wed, Jun 29, 2005 at 10:48:02PM +0300, Martin Vermeer wrote: > > On Wed, Jun 29, 2005 at 10:46:18AM +0300, Martin Vermeer wrote: > > > > ... > > > > > Here is the patch, armed with comments for posterity. > > > > And here a bet

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-07-01 Thread Andre Poenitz
On Wed, Jun 29, 2005 at 10:48:02PM +0300, Martin Vermeer wrote: > On Wed, Jun 29, 2005 at 10:46:18AM +0300, Martin Vermeer wrote: > > ... > > > Here is the patch, armed with comments for posterity. > > And here a better one. I don't agree with the idea of storing row and column in the cursor.

Re: [New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-06-30 Thread Martin Vermeer
On Wed, Jun 29, 2005 at 10:48:02PM +0300, Martin Vermeer wrote: > On Wed, Jun 29, 2005 at 10:46:18AM +0300, Martin Vermeer wrote: > > ... > > > Here is the patch, armed with comments for posterity. > > And here a better one. > Committed. - Martin pgpqvsA7C9RHy.pgp Description: PGP signa

[New patch] Re: 1.4 crash edit->rows&columns->copy column

2005-06-29 Thread Martin Vermeer
On Wed, Jun 29, 2005 at 10:46:18AM +0300, Martin Vermeer wrote: ... > Here is the patch, armed with comments for posterity. And here a better one. - Martin Index: cursor_slice.h === RCS file: /usr/local/lyx/cvsroot/lyx-devel/src