Hi,
With current CVS:
1. Start a new document. 2. Add some text. 3. Resize that text.
gives a load of assertion failures and then a segfault. After a cvs up -D 2004-04-19 everything's ok.
Cheers
Carl
Carl Hetherington wrote:
Hi,
With current CVS:
- Start a new document.
- Add some text.
- Resize that text.
gives a load of assertion failures and then a segfault. After a cvs up -D 2004-04-19 everything's ok.
It appears to be the new SPSelection code, which is _Very_ alpha at the moment. It would be most efficient to wait until mental gets up in the morning I think. I'm digging, and it seems that selection->desktop is null. but I haven't worked out why.
njh
On Tue, 20 Apr 2004, Nathan Hurst wrote:
Carl Hetherington wrote:
Hi,
With current CVS:
- Start a new document.
- Add some text.
- Resize that text.
gives a load of assertion failures and then a segfault. After a cvs up -D 2004-04-19 everything's ok.
It appears to be the new SPSelection code, which is _Very_ alpha at the moment. It would be most efficient to wait until mental gets up in the morning I think. I'm digging, and it seems that selection->desktop is null. but I haven't worked out why.
This seems to fix it:
Index: src/text-context.cpp =================================================================== RCS file: /cvsroot/inkscape/inkscape/src/text-context.cpp,v retrieving revision 1.29 diff -u -r1.29 text-context.cpp --- src/text-context.cpp 20 Apr 2004 06:07:27 -0000 1.29 +++ src/text-context.cpp 20 Apr 2004 13:43:25 -0000 @@ -191,6 +191,9 @@ { SPTextContext *tc = SP_TEXT_CONTEXT (ec);
+ tc->sel_changed_connection.disconnect(); + tc->sel_modified_connection.disconnect(); + sp_text_context_forget_text (SP_TEXT_CONTEXT (ec));
if (tc->imc) {
Mental, could you check whether this is appropriate? I will commit if so.
Thanks
Carl
On Tue, 2004-04-20 at 09:45, Carl Hetherington wrote:
This seems to fix it:
Index: src/text-context.cpp
RCS file: /cvsroot/inkscape/inkscape/src/text-context.cpp,v retrieving revision 1.29 diff -u -r1.29 text-context.cpp --- src/text-context.cpp 20 Apr 2004 06:07:27 -0000 1.29 +++ src/text-context.cpp 20 Apr 2004 13:43:25 -0000 @@ -191,6 +191,9 @@ { SPTextContext *tc = SP_TEXT_CONTEXT (ec);
tc->sel_changed_connection.disconnect();
tc->sel_modified_connection.disconnect();
sp_text_context_forget_text (SP_TEXT_CONTEXT (ec));
if (tc->imc) {
Mental, could you check whether this is appropriate? I will commit if so.
That looks appropriate. Go for it.
-mental
participants (3)
-
Carl Hetherington
-
MenTaLguY
-
Nathan Hurst