Hi all,
As I'm researching my book on the guts of LibreOffice, I notice that tools predates the SAL. In
fact it's a bit of a hodge-lodge of files, many of which look like they should probably migrated
into other modules.
I cleaned up ErrorHandler, but I thought I'd ask about the other headers here:
- b3dtrans.hxx - I'm assuming the b3d bit means basegfx 3D transformations. I've submitted
https://gerrit.libreoffice.org/#/c/37030/ to move it into basegfx but unfortunately it's failing o
Windows and I don't currently have a working system to determine why the linker is erroring out.
- bigint.hxx and fract.hxx - seems to be more appropriate to the sal module, possibly the RTL?
Great to get thoughts of others on this one
- color.hxx and colordata.hxx - should that migrate to basegfx also?
- gen.hxx - includes Pair (shouldn't that be deprecated to std::pair?), Point, Size, Range,
Selection and Rectangle
- poly.hxx - polygons
- line.hxx - lines
gen.hxx, lines.hxx and poly.hxx all are drawing primitives, wouldn't these be better off in basegfx?
- config.hxx - where would this go? I was thinking the RTL, but it also seems like something the
VLC might do...
- contr.hxx - seems better suited in the svx module...
- cpuid.hxx - checks for SSE2 support, surely something for the SAL?
- time.hxx, date.hxx, datetime.hxx and datetimeutils.hxx - these all seem to be better suited to
the SAL, and actually should we consider moving to chrono?
- debug.hxx and diagnose_ex.hxx - seems more suited to the RTL...
-extendApplicationEnvironment - seems to be a candidate for the OSL...
- fldunit.hxx and fontenum.hxx - both seems to be more appropriate in the VCL
There are quite a few headers, but as you can see most look to be more appropriately moved to
another module. This would help streamline our module dependencies...
Thoughts?
Chris
Sent from my iPhone
Context
- Moving away from tools module · Chris Sherlock
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.