Re: [PATCH] Re: input for fdo#45779 from a basegfx knowledgeable person needed

2012-05-18 Thread Pierre-André Jacquod
Hello, On 05/03/2012 11:30 AM, Thorsten Behrens wrote: So what I suggest is a more defensive fix (or some larger review across the code is in order): make GetLineArrow() return the B2DPolyPolygon right away, and have *that one* be empty (i.e. not a Since I do not intend to become a specialist

Re: [PATCH] Re: input for fdo#45779 from a basegfx knowledgeable person needed

2012-05-03 Thread Thorsten Behrens
Pierre-André Jacquod wrote: > back again after a while. > Hi Pierre-André, welcome back! :) > It happens that basegfx::GetLineArrow(...) (also defined within > msdffimp.cxx, line 1102) does not create a valid polygon when > eLineEnd has the value mso_lineNoEnd... > In the switch(eLineEnd), this g

[PATCH] Re: input for fdo#45779 from a basegfx knowledgeable person needed

2012-04-30 Thread Pierre-André Jacquod
Hello, back again after a while. On 02/15/2012 11:30 AM, Thorsten Behrens wrote: Fixed with d37abad97d72bae0fd0269de12e94c7a7d3fd7e1 - but, if you like, would be cool to chase down why in the first place the ppt import creates polygons with empty sub-paths, that looks like a worthwhile optimiza