[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Add and use a Transform data type (issue 344970043 by address@hidden

From: dak
Subject: Re: Add and use a Transform data type (issue 344970043 by address@hidden)
Date: Sun, 17 Jun 2018 04:05:36 -0700

On 2018/06/17 10:54:38, dak wrote:
Right.  I haven't rebased this patch yet, but I naturally will have to
load up
another version for review now that the fix is in master.

I actually lean towards doing this in a manner similar to your original
approach, namely just using the stencil rotation call.  Basically I'd
want to completely replace both sten and s with a rotated copy.  This
will make a difference for later uses of s, notably

  // we use the bounding box if there are no boxes
  // FIXME: Rotation?
  if (!boxes.size () && !buildings.size ())
    boxes.push_back (Box (s->extent (X_AXIS), s->extent (Y_AXIS)));

The question is whether using the rotated version here is supposedly an
improvement, detrimental or "don't care".  Regarding the "relative"
implementation of the origin location, the bounding boxes resulting from
empty extents are likely to be garbage (likely NaN) after rotation.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]