On Sun, Aug 03, 2008 at 06:15:51AM -0500, Aaron Spike wrote:
Diederik van Lierop wrote:
J.B.C.Engelen@...1578... wrote:
I just started mass converting to Geom::Point, but then it struck me that this will conflict with many changes other people are creating.
That was my first thought also when I read the subject of your mail. Maybe this should wait until just after the next release?
Perhaps my opinion isn't relevant right now because I'm not actively working on anything but... The focus of this release cycle is nominally on refactoring. I we're postponing refactoring something is wrong. With a mass conversion there will be pain once for everyone. Without a mass conversion NR::Point will live on forever as it is continually added to the code base again and again. I say convert things and then remove NR::Point along with the temptation to use it. Alternatively give people a week to do a SVN diff and update what they are working on themselves and after the week convert everything else. Feel free to disagree. :-)
Agreed with all the above. The purpose of this release is refactoring, so refactor away.
Bryce