<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><br><div class="gmail_quote"><div dir="ltr">On Sat, 8 Dec 2018 at 20:25, Regina Henschel <<a href="mailto:rb.henschel@t-online.de">rb.henschel@t-online.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Problem A<br>
The current implementation has a method GetPoint, which returns a <br>
tools::Point; and such has integer coordinates. This introduces <br><br></blockquote><div><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">I would say you should change GetPoint() to return a basegfx::B2Point. You can't have two methods with the same name with different return types, and this is a place where it clearly should be returning a more accurate result.</div><div class="gmail_default" style="font-family:tahoma,sans-serif"></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Problem B<br>to createPolygonFromEllipseSegment to allow generating of <br>
counter-clockwise arcs. Which way to go?<br>
<br></blockquote><div><span class="gmail_default" style="font-family:tahoma,sans-serif">You should add a parameter here. Either a boolean or an enum param, something like 'enum class Direction { Clockwise, CounterClockwise }'</span></div><div><span class="gmail_default" style="font-family:tahoma,sans-serif"></span> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Problem C<br>the path is ill-structured and the first parameter point is consumed by <br>
moveTo? Keep the direction or toggle it?<br>
<br></blockquote><div><span class="gmail_default" style="font-family:tahoma,sans-serif">With ill-structured data, I would say that you should just 'not crash' and error out at the earlier opportunity. There is no need to try doing something sensible with such data unless necessary for some kind of compaibility.</span></div><div><span class="gmail_default" style="font-family:tahoma,sans-serif"></span> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Problem D<br>
How to make a unit test for such patches <br>
(<a href="https://gerrit.libreoffice.org/#/c/64704/1" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/#/c/64704/1</a> is another one)? It would <br>
need to compare a bad rendering with a correct one and it would need to <br><br></blockquote><div><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">For this kind of thing, one approach is to query the output programmatically to see that it is correct. Other times we dump the output using dumpAsXml() and query the output using XPath.</div><div class="gmail_default" style="font-family:tahoma,sans-serif"></div></div></div>