[GitHub] [openoffice] pedlino edited a comment on pull request #89: Use std::vector instead of fixed-size array of cffLocal objects

[GitBox <gi...@apache.org>]

pedlino edited a comment on pull request #89:
URL: https://github.com/apache/openoffice/pull/89#issuecomment-695216940


   > I am feeling a bit stupid while typing this, like I am reinventing the wheel using the wrong tools, but please understand I am only trying to help.
   
   Never feel stupid for trying to help! If the wheel was already invented but you can't find it, it is certainly not your fault ;)
   
   Such an ODT would help. I think the description is not necessary. What is needed is a screenshot before and after the patch is applied. If visually there is no loss of characters or loss of quality of rendering then the patch can be accepted.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org