Re: curent cvs crash

2003-03-08 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Sat, Mar 08, 2003 at 06:29:00PM +0100, Lars Gullik Bj?nnes wrote: | | > | That is the old behaviour and that's what the patch does. | > | > Yes, but at the same time we want cleanups to move forward. | | Then help out a poor brother. I don't understan

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 06:29:00PM +0100, Lars Gullik Bj?nnes wrote: > | That is the old behaviour and that's what the patch does. > > Yes, but at the same time we want cleanups to move forward. Then help out a poor brother. I don't understand the code that calls labelhfill john

Re: curent cvs crash

2003-03-08 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Sat, Mar 08, 2003 at 05:57:19PM +0100, Lars Gullik Bj?nnes wrote: | | > | I give up. Lars, a little help here ... the cursor behaves oddly on an | > | empty list par as well I noticed. The patch below restores the old | > | behaviour if that's all you

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 05:57:19PM +0100, Lars Gullik Bj?nnes wrote: > | I give up. Lars, a little help here ... the cursor behaves oddly on an > | empty list par as well I noticed. The patch below restores the old > | behaviour if that's all you want > > I want it to not abort and not crash. T

Re: curent cvs crash

2003-03-08 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Sat, Mar 08, 2003 at 04:04:25PM +, John Levon wrote: | | > I'm bad ... I changed an Asger (aka fix-up-the-bug) into a real | > Assert to see what would happen, but I forgot to leave a comment there. | > Now we have a simple testcase which is fab. W

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 04:04:25PM +, John Levon wrote: > I'm bad ... I changed an Asger (aka fix-up-the-bug) into a real > Assert to see what would happen, but I forgot to leave a comment there. > Now we have a simple testcase which is fab. We can actually kill the > bug not do the moral equ

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 05:02:49PM +0100, Lars Gullik Bj?nnes wrote: > :-) I probably should have said... Obvioulsy the assert triggers. > and that would be equally silly. I'm bad ... I changed an Asger (aka fix-up-the-bug) into a real Assert to see what would happen, but I forgot to leave a com

Re: curent cvs crash

2003-03-08 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Sat, Mar 08, 2003 at 04:52:53PM +0100, Lars Gullik Bj?nnes wrote: | | > | I'm looking but beginningofMainBody() really shouldn't be returning < 0 | > | now should it ? | > | > Perhaps... but obviously it does. | | Silly thing to say. It's returning ==

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 04:52:53PM +0100, Lars Gullik Bj?nnes wrote: > | I'm looking but beginningofMainBody() really shouldn't be returning < 0 > | now should it ? > > Perhaps... but obviously it does. Silly thing to say. It's returning == 0. Now what I don't get is that we're calling this cod

Re: curent cvs crash

2003-03-08 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Sat, Mar 08, 2003 at 04:43:43PM +0100, Lars Gullik Bj?nnes wrote: | | > 1.293(levon26-Feb-03): lyx::Assert(last > 0); | | I'm looking but beginningofMainBody() really shouldn't be returning < 0 | now should it ? Perhaps... but obvious

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 04:43:43PM +0100, Lars Gullik Bj?nnes wrote: > pos_type last = row.par()->beginningOfMainBody(); > > lyx::Assert(last > 0); > ^^ Bleh, OK, so we're calling labelFill on emptyPar. Now before, we ended up with a last of -1. We got

Re: curent cvs crash

2003-03-08 Thread John Levon
On Sat, Mar 08, 2003 at 04:43:43PM +0100, Lars Gullik Bj?nnes wrote: > 1.293(levon26-Feb-03): lyx::Assert(last > 0); I'm looking but beginningofMainBody() really shouldn't be returning < 0 now should it ? john

Re: curent cvs crash

2003-03-08 Thread Lars Gullik Bjønnes
Alfredo Braunstein <[EMAIL PROTECTED]> writes: | File->New | select "List" from the layout dropdown -> crash | Cannot get a backtrace though, my gdb is still looping. | I get it with both frontends. The crash is here: text.C: // returns the minimum space a manual label needs on the screen in pi