Re: Improve WriteCustomGeometry in oox export; review of patch

2022-03-25 Thread Miklos Vajna
Hi Regina,

On Thu, Mar 24, 2022 at 07:33:24PM +0100, Regina Henschel 
 wrote:
> The commit message is quite detailed. If something is still unclear, please
> send me a mail.

Not only that, but the various test files you added in the tests also
helped to understand the problem you solve. Thanks for that.

> BTW, Jenkins fails for a reason I don't know. But I want to first
> incorporate your suggestions before I start a next try with Jenkins.

It was just a one-off failure, it went away after a "resume".

Regards,

Miklos


Improve WriteCustomGeometry in oox export; review of patch

2022-03-24 Thread Regina Henschel

Hi Miklos, hi all,

the patch in https://gerrit.libreoffice.org/c/core/+/131837 is now ready 
for review.


It is a change in the way a custGeom element is generated. I have 
dropped the export via PolyPolygon. Instead our path commands are 
directly mapped to oox path commands. That is a larger change, so it 
would be good, if you find time to review it.


The commit message is quite detailed. If something is still unclear, 
please send me a mail.


BTW, Jenkins fails for a reason I don't know. But I want to first 
incorporate your suggestions before I start a next try with Jenkins.


Kind regards,
Regina