Re: [DIAGNOSIS] Re: crash in trunk

2011-04-24 Thread Richard Heck
On 04/24/2011 06:02 AM, Edwin Leuven wrote: Richard Heck: What the usual calls to updateBuffer() do is always make the macro context a ParIterator that points to the first paragraph in the current inset. (See e.g. the calls at InsetText:682,695.) ok. makes me wonder though why it is necessary t

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-24 Thread Edwin Leuven
Richard Heck : > What the usual calls to updateBuffer() do is always > make the macro context a ParIterator that points to > the first paragraph in the current inset. (See e.g. > the calls at InsetText:682,695.) ok. makes me wonder though why it is necessary to set the macro context explicitly. >

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Tommaso Cucinotta
Il 23/04/2011 19:52, Edwin Leuven ha scritto: Richard Heck wrote: I guess the other idea would be to recurse through all the insets. attached, and no more crashes. i think i prefer this solution (although i am not sure we need to add the code to insettext) ed. FYI, I just added a test-case

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Richard Heck
On 04/23/2011 02:07 PM, John McCabe-Dansted wrote: On Sat, Apr 23, 2011 at 11:45 PM, Edwin Leuven wrote: ..\..\..\lyx-devel\src\support\lassert.cpp(21): ASSERTION false VIOLATED IN ..\..\lyx-devel\src\CursorSlice.cpp:188 In case this is relevant, this assertion also shows up in: http://gmat

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Richard Heck
Regarding the x2.patch, the problem is that par_iterator_begin() does not give us a "complete" DocIterator (one starting with the Buffer's own InsetText, etc), which is what we need. I am constantly running into this problem! Regarding the other patch, adding the code to InsetText just seeme

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread John McCabe-Dansted
On Sat, Apr 23, 2011 at 11:45 PM, Edwin Leuven wrote: > ..\..\..\lyx-devel\src\support\lassert.cpp(21): ASSERTION false > VIOLATED IN ..\..\lyx-devel\src\CursorSlice.cpp:188 In case this is relevant, this assertion also shows up in: http://gmatht.homelinux.net/xp/keytest/html_out/out/t18//html/

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Edwin Leuven
Richard Heck wrote: > I guess the other idea would be to recurse > through all the insets. attached, and no more crashes. i think i prefer this solution (although i am not sure we need to add the code to insettext) ed. x3.patch Description: Binary data

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Edwin Leuven
Richard Heck wrote: > Can you post this version of the patch > and I'll try to have a look? attached > I'm wondering where these comparisons are > being made and whether we need to do > something to the cursor. But I'm really > just guessing here. very frustating this poking in the dark... > I

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Richard Heck
On 04/23/2011 11:45 AM, Edwin Leuven wrote: I wonder if we could try: ParagraphList::const_iterator pit = tail.getText(0)->paragraphs().begin(); buffer().updateBuffer(pit, OutputUpdate); i am not sure i completely follow you, but when i use buffer().updateBuffer(par_iterator_begin(tai

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Richard Heck
On 04/23/2011 11:45 AM, Edwin Leuven wrote: I wonder if we could try: ParagraphList::const_iterator pit = tail.getText(0)->paragraphs().begin(); buffer().updateBuffer(pit, OutputUpdate); i am not sure i completely follow you, but when i use buffer().updateBuffer(par_iterator_begin(tai

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Edwin Leuven
Richard Heck wrote: > My main comment would be that I suspect, but of course > couldn't verify, due to the crashes, that the same > problem exists in the TexRow() output routine, i checked that previewing the document and having the source view pane open worked my guess was that things are ok as

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Richard Heck
On 04/23/2011 06:56 AM, Edwin Leuven wrote: Richard Heck wrote: Answer: No, probably not, because it's hard to see how to do this without making copies, which would just give us the same problem back again. the attached avoids the crash for me by setting the macrocontext for nested insets (and

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-23 Thread Edwin Leuven
Richard Heck wrote: > Answer: No, probably not, because it's hard to see > how to do this without making copies, which would > just give us the same problem back again. the attached avoids the crash for me by setting the macrocontext for nested insets (and only doing the metrics call on the tail;

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-22 Thread Richard Heck
On 04/22/2011 10:42 PM, Richard Heck wrote: On 04/22/2011 05:24 PM, Edwin Leuven wrote: Richard Heck wrote: I think I know what's happening here. sounds reasonable indeed We might somehow need to call updateBuffer() on these new insets. or find a way of calculating the width of the insette

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-22 Thread Richard Heck
On 04/22/2011 05:24 PM, Edwin Leuven wrote: Richard Heck wrote: I think I know what's happening here. sounds reasonable indeed We might somehow need to call updateBuffer() on these new insets. or find a way of calculating the width of the insettext before and after the decimal separator wit

Re: [DIAGNOSIS] Re: crash in trunk

2011-04-22 Thread Edwin Leuven
Richard Heck wrote: > I think I know what's happening here. sounds reasonable indeed > We might somehow need to call updateBuffer() > on these new insets. or find a way of calculating the width of the insettext before and after the decimal separator without making a copy, chopping it into two a

[DIAGNOSIS] Re: crash in trunk

2011-04-22 Thread Richard Heck
On 04/22/2011 02:17 PM, Richard Heck wrote: On 04/22/2011 01:46 PM, Edwin Leuven wrote: Richard Heck wrote: Same crash with a footnote there. So it looks as if for some reason insets inside cells are not getting their macro context set properly. which is done in lines 3453-3473 of insettabula

Re: crash in trunk

2011-04-22 Thread Richard Heck
On 04/22/2011 01:46 PM, Edwin Leuven wrote: Richard Heck wrote: Same crash with a footnote there. So it looks as if for some reason insets inside cells are not getting their macro context set properly. which is done in lines 3453-3473 of insettabular.cpp: if (tabular.getAlignment(cell) == LY

Re: crash in trunk

2011-04-22 Thread Edwin Leuven
Richard Heck wrote: > Same crash with a footnote there. So it looks > as if for some reason insets inside cells are not > getting their macro context set properly. which is done in lines 3453-3473 of insettabular.cpp: if (tabular.getAlignment(cell) == LYX_ALIGN_DECIMAL) { // make a copy

Re: crash in trunk

2011-04-22 Thread Richard Heck
On 04/22/2011 01:36 PM, Richard Heck wrote: On 04/22/2011 12:01 PM, Edwin Leuven wrote: assert in textmetrics.cpp line 395 when i scoll down (or use toc) to table 5 in this doc: http://dl.dropbox.com/u/359550/peerng.lyx a broken document, just when i switch to trunk for real work :( The caus

Re: crash in trunk

2011-04-22 Thread Richard Heck
On 04/22/2011 12:01 PM, Edwin Leuven wrote: assert in textmetrics.cpp line 395 when i scoll down (or use toc) to table 5 in this doc: http://dl.dropbox.com/u/359550/peerng.lyx a broken document, just when i switch to trunk for real work :( The cause is the InsetScript in that table. If you re

Re: Crash in trunk

2008-02-29 Thread Pavel Sanda
> Pavel Sanda wrote: >> Open math manual. >> Ctrl+F and Find next "Because all" >> Toggle Search backwards >> Twice push Find next button >> Kaboom! >> > Can't reproduce this one difficult to reproduce, when its already fixed :D pavel

Re: Crash in trunk

2008-02-29 Thread Richard Heck
Pavel Sanda wrote: Open math manual. Ctrl+F and Find next "Because all" Toggle Search backwards Twice push Find next button Kaboom! Can't reproduce this one [EMAIL PROTECTED] ~]$ uname -r && rpm -q qt4 2.6.23.15-137.fc8 qt4-4.3.3-1.fc8 rh

Re: Crash in trunk

2008-02-29 Thread Pavel Sanda
> Fixed in r23354. thanks pavel

Re: Crash in trunk

2008-02-29 Thread Stefan Schimanski
Fixed in r23354. Stefan Am 29.02.2008 um 16:02 schrieb Pavel Sanda: Open math manual. Ctrl+F and Find next "Because all" Toggle Search backwards Twice push Find next button Kaboom!

Re: Crash in trunk

2008-02-29 Thread Stefan Schimanski
Backtraces make life easier: #4 0x95f756f9 in __gnu_debug::_Error_formatter::_M_error () #5 0x00114512 in __gnu_debug ::_Safe_iterator<__gnu_cxx::__normal_iterator__gnu_norm::vector > >, __gnu_debug_def::vector > >::operator-> (this=0xbfffce04) at safe_iterator.h:191 #6 0x00113182 in ly

Re: Crash in trunk with Completion framework

2008-02-25 Thread Stefan Schimanski
Am 25.02.2008 um 11:25 schrieb Abdelrazak Younes: Stefan Schimanski wrote: What did you do to make this happen? 1) new document 2) type "lalalala l" 3) type "a" Works fine here. Even using stdlib-debug. Stefan

Re: Crash in trunk with Completion framework

2008-02-25 Thread Abdelrazak Younes
Stefan Schimanski wrote: What did you do to make this happen? 1) new document 2) type "lalalala l" 3) type "a" Abdel.

Re: Crash in trunk with Completion framework

2008-02-25 Thread Stefan Schimanski
lyx.exe!std::char_traits::assign(unsigned long & _Left=3452816845, const unsigned long & _Right=) Line 173 + 0x6 bytes C++ lyx.exe!std::char_traits::_Copy_s(unsigned long * _First1=0x10560040, unsigned int _Dest_size=28716311, const unsigned long * _First2=0x01b81000, unsigned int _Count

Re: Crash in trunk related to layout, InsetSpace or SpecialChar

2008-02-20 Thread Richard Heck
Abdelrazak Younes wrote: Abdelrazak Younes wrote: This happens while trying to open any of the help documents. Richard maybe? lyx.exe!lyx::Paragraph::isFreeSpacing() Line 2373 + 0x27 bytesC++ By the way d->inset_owner_ is an InsetFoot here if that makes any difference. Yes, p

Re: Crash in trunk related to layout, InsetSpace or SpecialChar

2008-02-20 Thread Abdelrazak Younes
Abdelrazak Younes wrote: This happens while trying to open any of the help documents. Richard maybe? lyx.exe!lyx::Paragraph::isFreeSpacing() Line 2373 + 0x27 bytesC++ By the way d->inset_owner_ is an InsetFoot here if that makes any difference. Abdel.

Re: Crash in Trunk

2008-02-19 Thread Abdelrazak Younes
Andre Poenitz wrote: On Tue, Feb 19, 2008 at 08:32:30AM +0100, Abdelrazak Younes wrote: rgheck wrote: Do something to lyx2lyx to cause a syntax error. Now try to open a file that needs converting. BOOM! Does this help? Of course inset_iterator_begin() should not crash to begin with. Author:

Re: Crash in Trunk

2008-02-19 Thread Andre Poenitz
On Tue, Feb 19, 2008 at 08:32:30AM +0100, Abdelrazak Younes wrote: > rgheck wrote: >> Do something to lyx2lyx to cause a syntax error. Now try to open a file >> that needs converting. BOOM! > > Does this help? Of course inset_iterator_begin() should not crash to begin > with. > > Author: younes >

Re: Crash in Trunk

2008-02-18 Thread Abdelrazak Younes
rgheck wrote: Do something to lyx2lyx to cause a syntax error. Now try to open a file that needs converting. BOOM! Does this help? Of course inset_iterator_begin() should not crash to begin with. Author: younes Date: Tue Feb 19 08:31:02 2008 New Revision: 23063 URL: http://www.lyx.org/tra

Re: Crash in Trunk

2008-01-10 Thread Abdelrazak Younes
rgheck wrote: Abdelrazak Younes wrote: Please see if this helps: URL: http://www.lyx.org/trac/changeset/22473 I'll test it tomorrow. Time for bed Fine. Time to really work for me ;-) Abdel.

Re: Crash in Trunk

2008-01-10 Thread rgheck
Abdelrazak Younes wrote: Please see if this helps: URL: http://www.lyx.org/trac/changeset/22473 I'll test it tomorrow. Time for bed rh

Re: Crash in Trunk

2008-01-10 Thread Abdelrazak Younes
rgheck wrote: Abdelrazak Younes wrote: rgheck wrote: Abdelrazak Younes wrote: rgheck wrote: Happened while it was just sitting there. I've seen this several times. But this doesn't look like a sane backtrace, did you forgot to type bt? As non-sane as it may look, that's exactly what I go

Re: Crash in Trunk

2008-01-10 Thread rgheck
Abdelrazak Younes wrote: rgheck wrote: Abdelrazak Younes wrote: rgheck wrote: Happened while it was just sitting there. I've seen this several times. But this doesn't look like a sane backtrace, did you forgot to type bt? As non-sane as it may look, that's exactly what I got when I typed

Re: Crash in Trunk

2008-01-10 Thread Abdelrazak Younes
rgheck wrote: Abdelrazak Younes wrote: rgheck wrote: Happened while it was just sitting there. I've seen this several times. But this doesn't look like a sane backtrace, did you forgot to type bt? As non-sane as it may look, that's exactly what I got when I typed bt. OK. It seems that t

Re: Crash in Trunk

2008-01-09 Thread rgheck
Abdelrazak Younes wrote: rgheck wrote: Happened while it was just sitting there. I've seen this several times. But this doesn't look like a sane backtrace, did you forgot to type bt? As non-sane as it may look, that's exactly what I got when I typed bt. It seems that these scrollbar actions

Re: Crash in Trunk

2008-01-09 Thread Abdelrazak Younes
rgheck wrote: Happened while it was just sitting there. I've seen this several times. But this doesn't look like a sane backtrace, did you forgot to type bt? Abdel.

Re: crash in trunk (depm related)

2007-12-27 Thread Jürgen Spitzmüller
Abdelrazak Younes wrote: > Yes, that was it. confirmed. > I don't know if this is applicable to 1.5. The problem does not occur in branch. Jürgen

Re: crash in trunk (depm related)

2007-12-27 Thread Abdelrazak Younes
Abdelrazak Younes wrote: Jürgen Spitzmüller wrote: - new document - type "abc" - return (generate empty paragraph) - click between a and b with the mouse I think this is because the BufferView Cursor is not up to date at the time of the drawing (in BufferView::checkDepm()). But it is weird th

Re: crash in trunk (depm related)

2007-12-27 Thread Abdelrazak Younes
Jürgen Spitzmüller wrote: - new document - type "abc" - return (generate empty paragraph) - click between a and b with the mouse I think this is because the BufferView Cursor is not up to date at the time of the drawing (in BufferView::checkDepm()). But it is weird that 1.5 is not affected by

Re: crash in trunk (blinkingcursor)

2007-10-11 Thread Jean-Marc Lasgouttes
"Leuven, E." <[EMAIL PROTECTED]> writes: > PS if you wonder why i am fiddling with ctrl-w: i was tricked by my > french (azerty) keyboard with US international bindings... I know this feeling... JMarc

Re: crash in trunk (blinkingcursor)

2007-10-11 Thread Abdelrazak Younes
Leuven, E. wrote: ctrl-n ctrl-w BOOM! backtrace below Oups my fault... side effect of my fix to the disappearing cursor. I'll correct that. Abdel.

RE: crash in trunk (preferences->converters)

2007-09-19 Thread Leuven, E.
this http://www.lyx.org/trac/changeset/20357 should take care of it... -Original Message- From: Jürgen Spitzmüller [mailto:[EMAIL PROTECTED] Sent: Tue 9/18/07 11:50 To: LyX Devel Subject: crash in trunk (preferences->converters) - Go to Preferences->Converters - Select a converter, sa

RE: crash in trunk (preferences->converters)

2007-09-18 Thread Leuven, E.
i'll investigate... -Original Message- From: Jürgen Spitzmüller [mailto:[EMAIL PROTECTED] Sent: Tue 9/18/07 11:50 To: LyX Devel Subject: crash in trunk (preferences->converters) - Go to Preferences->Converters - Select a converter, say LaTeX (plain) -> DVI - Modify this converter - Hit

Re: crash in trunk

2007-08-31 Thread Abdelrazak Younes
Bernhard Roider wrote: 1. new document 2. insert a (figure) float 3. press the left cursor key twice (OR press the up key once) 4. press he right cursor key I've committed a fix for this in rev 19919: Author: younes Date: Thu Aug 30 18:48:47 2007 New Revision: 19919 URL: http://www.lyx.org/tr

Re: crash in trunk

2006-09-20 Thread Abdelrazak Younes
Edwin Leuven wrote: Abdelrazak Younes wrote: Edwin Leuven wrote: BOOM! Yes, this is an old one. selecting (multiple) cells is not possible either Even inserting a single character doesn't work ;-/ this kinda sucks... Yep. Abdel.

Re: crash in trunk

2006-09-20 Thread Edwin Leuven
Abdelrazak Younes wrote: Edwin Leuven wrote: BOOM! Yes, this is an old one. selecting (multiple) cells is not possible either this kinda sucks...

Re: crash in trunk

2006-09-20 Thread Abdelrazak Younes
Edwin Leuven wrote: insert table put cursor in cell shift + arrow down BOOM! Yes, this is an old one. The problem is that the CoordCache is not correctly reconstructed in the updateMetrics within the table. Some missing points are not being re-added after the CoordCache has been cleared in B