Re: Dia VDX plug-in 0.6

[Ian Redfern <Ian Redfern LogicaCMG com>]

OK - found it. It wasn't correctly inheriting the existence of Masters from a group parent.

The fix is:

--- vdx-import.c        (revision 185)
+++ vdx-import.c        (working copy)
@@ -1866,7 +1866,7 @@
                 if (!theShape->Master) 
                 { 
                     theShape->Master = Shape->Master; 
-                    theShape->Master_exists = TRUE;
+                    theShape->Master_exists = Shape->Master_exists;
                 }
                 members = vdx_plot_shape(theShape, members, XForm, theDoc);
             }

I've created a 0.6.1 with just this fix.

Thanks,
Ian Redfern.

On Wed, 2006-11-15 at 08:06 +0100, Lars Clausen wrote:
> On Tue, 2006-11-14 at 21:40 +0100, Lars Clausen wrote:
> > On Tue, 2006-11-14 at 10:01 +0000, Ian Redfern wrote:
> > > Version 0.6 of the Dia VDX plug-in can now be found at
> > > http://www.redferni.uklinux.net/dia/vdx/
> > 
> > I shall update the CVS as soon as I can (can't compile right now due to
> > changes I'm making in lib/textline.c).
>
> I ran the new code on the vdxtosvg examples in the samples dir, and
> ended up with a crash after:
>
> ** ERROR **: file vdx-import.c: line 508 (get_master_shape): assertion
> failed: (master < theDoc->Masters->len)
>
> So I'm afraid I cannot commit it yet.
>
> -Lars
>
> _______________________________________________
> Dia-list mailing list
> Dia-list gnome org
> http://mail.gnome.org/mailman/listinfo/dia-list
> FAQ at http://www.gnome.org/projects/dia/faq.html
> Main page at http://www.gnome.org/projects/dia

Attachment: signature.asc
Description: This is a digitally signed message part