Hi Alex, Alexander Farrow wrote:
basegfx::B2DPoint <https://docs.libreoffice.org/basegfx/html/classbasegfx_1_1B2DPoint.html> does not have such a method. So calls to AdjustX(nHorzMove) are being replaced by something like setX(getX() + nHorzMove).
tools::Point is not exactly a role model for a nicely designed class, so I'd hesitate to take any literal bits out of that. ;) Many places using AdjustX() also call AdjustY() just the next line, so for that you could use basegfx::B2DPoint::operator+=(const B2DTuple&) If you believe you *must* use a single-component modifier, please come up with something that blends well with the b2dtuple/point concept. Also ideally, you will maintain some amount of internal consistency (e.g. compare how operator*, *= etc. all got added in farious flavours, such that math with b2dtuples works like the 'canonical' way when done with skalars or various forms of vectors). Cheers, -- Thorsten
Attachment:
signature.asc
Description: PGP signature