Re: Dia non-uniform scaling (aka "sub shapes") patch UPDATE (against svn rev. 3875)

[Lars Clausen <lars raeder dk>]

On Thu, 2008-01-10 at 15:47 +0100, Marcel Toele wrote:
> Hi guys,
>
> first of all, best wishes for the new year!
>
>
> With this post I present, to you, the latest update to the non-uniform
> scaling patch.
> This post is an update to the original posts:
>
> - The Preview Post:
> http://mail.gnome.org/archives/dia-list/2007-November/msg00023.html
> - The Request for Approval Post:
> http://mail.gnome.org/archives/dia-list/2007-November/msg00051.html
> - The Comments and Discussion follow-up Post:
> http://mail.gnome.org/archives/dia-list/2007-December/msg00005.html
>
> To summarize:
>
> 1. This patch allows to embed sub-shapes in a parent shape. The
> sub-shapes do not resize/distort when the parent shape
> is resized (i.e. they can resize independently).
> 2. Sub-shapes are "anchored" inside a parent shape. Supported anchors:
> Fixed (top/bottom/left/right) and Proportional. 
> 3. This patch fixes an outstanding bug in dia that prevented shapes
> for having a default size (i.e. the dreaded default 2x2cm
> default size is now only used as a fallback, for each shape a default
> size can now be specified in all units that dia supports). 
> 4. In order to prevent errors, the sub-shapes are trapped inside the
> parent shape.

Did you forget to include something in your patch?  The DiaUnitDef type
and the units array from widgets.c are not currently publicly available.
However, even when I make them so, your patch seems to have broken
shapes fatally, even the ones without subshapes.  The shapes simply do
not appear when I place them in the diagram.  Without your patch,
everything works fine.  Your patch also puts some "nan" values into
defaults.dia.

I'll commit the changes to make DiaUnitDef and units accessible (in
geometry.h), but the patch doesn't work.  

-Lars