Where to put a static method for common use in oox?

Tomaž Vajngerl quikee at gmail.com
Fri Dec 2 11:49:00 UTC 2022


Hi Regina,

On Fri, Dec 2, 2022 at 8:06 PM Regina Henschel <rb.henschel at t-online.de>
wrote:

> Hi Tomaž,
>
> isn't /core/include/oox/drawingml/shape.hxx a too wide scope? Would a
> new file in /core/oox/inc/drawingml/ work as well?
>

I wouldn't worry about one function (there are already many classes and
functions in "include" dir, which are only used in oox), but it's up to
you. If you create a new file for the header file, then please also add it
for the source file and move the implementation there.


> Kind regards,
> Regina
>
>
Tomaž
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice/attachments/20221202/788f684a/attachment.htm>


More information about the LibreOffice mailing list