From: Krzysztof K. <twe...@gm...> - 2012-02-25 19:32:29
|
Hello I saw that a Reflect transform was added to 2geom recently. It does not satisfy the requirements of TransformConcept: 1. Reflect * Reflect != Reflect (in fact there is no multiplication operator at all) 2. There is no identity element. Transforms are intended to be subgroups of Affine. Reflections are not a subgroup, so they should not be represented as a transform class. Instead, a function called Geom::reflection should return an Affine that represents the requested reflection. Regards, Krzysztof |