<html>
  <head>
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  </head>
  <body>
<div style="16px">Hi everybody,<br></div><div style="16px"><br></div><div style="16px">Recently for GSoC I have been looking into resolving<br></div><div style="16px"><a href="https://bugs.documentfoundation.org/show_bug.cgi?id=103322">"Use floating point for glyph positioning in VCL"</a><br></div><div style="16px"><br></div><div style="16px">This involves changing the types of variables from <span class="font" style="font-family: sans-serif, sans-serif;">tools::Point to basegfx::B2DPoint</span>.<br></div><div style="16px"><a href="https://docs.libreoffice.org/tools/html/classPoint.html"><span class="font" style="font-family: sans-serif, sans-serif;">tools::Point</span></a> has a method called <span class="font" style="font-family: sans-serif, sans-serif;">AdjustX</span><span class="font" style="font-family: sans-serif, sans-serif;">(long nHorzMove)</span><span class="font" style="font-family: sans-serif, sans-serif;"></span>, which adds nHorzMove to X:<br></div><div style="16px"><br></div><div style="16px"><a href="https://docs.libreoffice.org/basegfx/html/classbasegfx_1_1B2DPoint.html">basegfx::B2DPoint</a> does not have such a method. So calls to AdjustX(nHorzMove) are<br></div><div style="16px">being replaced by something like setX(getX() + nHorzMove).<br></div><div style="16px"><br></div><div style="16px">It works fine of course but I think AdjustX looks cleaner and it feels like a step<br></div><div style="16px">backwards when moving from tools::Point to basegfx::B2DPoint.<br></div><div style="16px"><br></div><div style="16px">An <a href="https://opengrok.libreoffice.org/search?project=core&refs=AdjustX">OpenGrok search</a> for AdjustX reveals it is used frequently in the code-base,<br></div><div style="16px">which suggests it is a method which developers have found useful in the past.<br></div><div style="16px"><br></div><div style="16px">If it is agreed that we should add such a method I would be happy to implement it.<br></div><div style="16px">Also for consistency we might want to add a similar method to other classes in<br></div><div style="16px">basegfx such as B2IPoint.<br></div><div style="16px"><br></div><div style="16px">With many thanks,<br></div><div style="16px">Alexander Farrow (IRC: Alexander Farrow)<br></div>  </body>
</html>