Hi Nicolai,
This is old code. Maybe at that time we did not have a great understanding of Athens. We use it to create path. For example:
TRBoxShape>>computePath canvas ifNil: [ ^ self ]. path := self athensCanvas createPath: [ :builder | builder absolute; moveTo: rectangle topLeft; lineTo: rectangle topRight; lineTo: rectangle bottomRight; lineTo: rectangle bottomLeft; lineTo: rectangle topLeft; closePath ]
We can easily rename the variable. But I am wondering whether we can remove that variable
Cheers, Alexandre
On Sep 12, 2016, at 6:56 AM, Nicolai Hess nicolaihess@gmail.com wrote:
During debugging the Athens-Crash I found something strange about TRCanvas (not responsible for the crash)
- TRCanvas canvas instance variable is not a AthensCairoCanvas at all but an AthensCairoSurface
- the canvas resp. surface is initialized with TRCanvas initial extent but a change on TRCanvas extent does not change the surfaces extent
- Is this canvas resp. surface actually used ? Do we only need it to manage some external resources (path/ image forms) ?
Can it be removed or at least renamed to something different.
regards nicolai _______________________________________________ Moose-dev mailing list Moose-dev@list.inf.unibe.ch https://www.list.inf.unibe.ch/listinfo/moose-dev