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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Regina,

On Fri, Dec 2, 2022 at 8:06 PM Regina Henschel <rb.henschel@xxxxxxxxxxx> 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ž

[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux