Review for "improve import/export of line styles"

Michael Weghorn m.weghorn at posteo.de
Mon Sep 2 06:18:48 UTC 2019


Hi Regina,

On 01/09/2019 19.44, Regina Henschel wrote:
> please have a look at https://gerrit.libreoffice.org/#/c/78372/
> 
> An important question is whether you agree that I have changed the
> import and export of our own line styles so that no prstDash line styles
> are used for them. Instead I export them as custDash elements and
> reconstruct them in the import.
> 
> I have not added tests yet. Please tell me, what you think would be good
> to test.
> 
> You know I always have a hard time with C++. So I welcome hints on how
> to implement something better.

I can't really say anything about your questions regarding line styles...

> 
> And technical question: What do I need to do locally in Git, so that the
> information, that the no longer needed file "lo_preset_dashes.odt" has
> to be deleted, is included in my commit?

You can use 'git rm <SOME_PATH>/lo_preset_dashes.odt' to mark that file
for deletion.

Kind regards,
Michael


More information about the LibreOffice mailing list