Patch to eliminate NRPoint and NRMatrix in favour of NR::*

[Jaspervdg]

This patch eliminates NRPoint and NRMatrix in favour of NR::Point and NR::Matrix (and friends).

A follow-up patch to eliminate NR::Point and NR::Matrix in favour of 2geom classes would be very nice (and if time permits I'll probably write one).

BTW, it does not (yet) eliminate overloads of functions for NR(::)Matrix* instead of NR::Matrix & (functions that were defined for both still are, except that the NRMatrix* is now an NR::Matrix*).

[Johan Engelen (johanengelen)]

Hi Jasper,

I think nobody is going to review a 162kb patch.
Do you have anything in particular you're uncertain about? Otherwise I think you should just commit it.

Thanks for your work!

[Jaspervdg (jaspervdg)]

Johan, I guess you're right that reviewing such a big patch isn't much fun, so I went ahead and committed the patch. It should be fine, as I did double-check it myself and it mostly consists of search/replace-like changes.

[Diederik van Lierop (mail-diedenrezi)]

No one complained, so I guess this one can be closed ;-)