Re: [PATCH] Change tracking hacking

2005-02-24 Thread Juergen Spitzmueller
Johnathan Burchill wrote: > The simplest way to reproduce it is: > 2. insert "a". > 3. track changes. > 4. cut "a" and paste it at the beginning of the par > 5. paste "a" at beginning again. > 6. paste "a" at beginning again ---> problem appears (4 "a"'s all marked > inserted). I have committed th

Re: [PATCH] Change tracking hacking

2005-02-24 Thread Lars Gullik Bjønnes
Juergen Spitzmueller <[EMAIL PROTECTED]> writes: | Juergen Spitzmueller wrote: >> As to the crash fix: can someone please have a look if this can be applied? >> (fixes the crash described in bug 1277). > | Lars? | (see http://marc.theaimsgroup.com/?l=lyx-devel&m=110894706811499&w=2 for the | patc

Re: [PATCH] Change tracking hacking

2005-02-24 Thread Juergen Spitzmueller
Juergen Spitzmueller wrote: > As to the crash fix: can someone please have a look if this can be applied? > (fixes the crash described in bug 1277). Lars? (see http://marc.theaimsgroup.com/?l=lyx-devel&m=110894706811499&w=2 for the patch). Jürgen

Re: [PATCH] Change tracking hacking

2005-02-22 Thread Juergen Spitzmueller
Johnathan Burchill wrote: > The simplest way to reproduce it is: > 2. insert "a". > 3. track changes. > 4. cut "a" and paste it at the beginning of the par > 5. paste "a" at beginning again. > 6. paste "a" at beginning again ---> problem appears (4 "a"'s all marked > inserted). > > It seems to be a

Re: [PATCH] Change tracking hacking

2005-02-21 Thread Johnathan Burchill
On Monday 21 February 2005 07:08, you wrote: > Johnathan Burchill wrote: > > I've attached the bug fix. There's no point in providing the > > break-paragraph patch until it's fully functional. If you want to play > > with it, see my first attachment in this thread. Whether or not you want > > to im

Re: [PATCH] Change tracking hacking

2005-02-21 Thread Juergen Spitzmueller
Johnathan Burchill wrote: > I've attached the bug fix. There's no point in providing the > break-paragraph patch until it's fully functional. If you want to play with > it, see my first attachment in this thread. Whether or not you want to > implement the functionality in 1.4.0, I'll work on it in

Re: [PATCH] Change tracking hacking

2005-02-20 Thread Johnathan Burchill
Lars Gullik Bjønnes wrote: >Juergen Spitzmueller <[EMAIL PROTECTED]> writes: > >| Lars Gullik Bjønnes wrote: >> >>> I am very happy to see bugs fixed. but I am a bit wary of adding >>> functionality at this stage. >> >| It's a bug fix really. The unability to break paragraphs is a severe (and >|

Re: [PATCH] Change tracking hacking

2005-02-15 Thread Lars Gullik Bjønnes
Juergen Spitzmueller <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: > >> I am very happy to see bugs fixed. but I am a bit wary of adding >> functionality at this stage. > | It's a bug fix really. The unability to break paragraphs is a severe (and | the biggest remaining AFAICS) limitat

Re: [PATCH] Change tracking hacking

2005-02-15 Thread John Levon
On Tue, Feb 15, 2005 at 03:21:22PM +0100, Juergen Spitzmueller wrote: > > I am very happy to see bugs fixed. but I am a bit wary of adding > > functionality at this stage. > > It's a bug fix really. The unability to break paragraphs is a severe (and > the biggest remaining AFAICS) limitation of

Re: [PATCH] Change tracking hacking

2005-02-15 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: > I am very happy to see bugs fixed. but I am a bit wary of adding > functionality at this stage. It's a bug fix really. The unability to break paragraphs is a severe (and the biggest remaining AFAICS) limitation of the change tracker (see bug 880). > Are you able to

Re: [PATCH] Change tracking hacking

2005-02-15 Thread Lars Gullik Bjønnes
Johnathan Burchill <[EMAIL PROTECTED]> writes: | Hi folks, > | Please find attached three files: a change-tracking patch, and a new inset | declaration and implementation. > | Use -p0 from the top-level source dir for applying the patch. | insetnewparagraph.{C,h} need to be placed in src/insets/

Re: [PATCH] Change tracking hacking

2005-02-14 Thread Juergen Spitzmueller
Johnathan Burchill wrote: > This patch addresses bug#1277, and adds functionality to ct. > The bug-fix addresses pasting while change-tracking. The main modification > is the "Change" parameter that has been added to moveItem(). > > The new functionality is to let the user break a paragraph within