tdf#42949 Fix IWYU warnings in include/tools/*

Found with bin/find-unneeded-includes
Only removal proposals are dealt with here.

Change-Id: I9dc920e9388f9e2aa4dcae1f0b3f7562d08f7f70
Reviewed-on: https://gerrit.libreoffice.org/61809
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>

21 files changed, 40 insertions, 32 deletions

diff --git a/include/IwyuFilter_include.yaml b/include/IwyuFilter_include.yaml
index d9b864fd6fb4..f31831b9298c 100644
--- a/include/IwyuFilter_include.yaml
+++ b/include/IwyuFilter_include.yaml
@@ -209,3 +209,26 @@ blacklist:
     - com/sun/star/lang/XServiceInfo.hpp
     - com/sun/star/rendering/XBezierPolyPolygon2D.hpp
     - com/sun/star/rendering/XLinePolyPolygon2D.hpp
+    include/tools/debug.hxx:
+    # Used behind #ifndef
+    - tools/toolsdllapi.h
+    include/tools/diagnose_ex.h:
+    # Needed for macros
+    - com/sun/star/lang/IllegalArgumentException.hpp
+    - com/sun/star/uno/RuntimeException.hpp
+    - rtl/ustring.hxx
+    include/tools/fontenum.hxx:
+    # Needed for template
+    - o3tl/typed_flags_set.hxx
+    include/tools/svlibrary.h:
+    # Used behind #ifndef
+    - osl/module.h
+    include/tools/poly.hxx:
+    # Needed for template
+    - o3tl/typed_flags_set.hxx
+    include/tools/stream.hxx:
+    # Needed for template
+    - o3tl/typed_flags_set.hxx
+    include/tools/urlobj.hxx:
+    # Needed for template
+    - o3tl/typed_flags_set.hxx
diff --git a/include/svx/svdobj.hxx b/include/svx/svdobj.hxx
index 028ee80bb2b0..630e8694f0d0 100644
--- a/include/svx/svdobj.hxx
+++ b/include/svx/svdobj.hxx
@@ -36,6 +36,7 @@
 #include <tools/weakbase.hxx>
 #include <tools/mapunit.hxx>
 #include <tools/gen.hxx>
+#include <osl/diagnose.h>
 #include <typeinfo>
 #include <functional>
 
diff --git a/include/tools/XmlWalker.hxx b/include/tools/XmlWalker.hxx
index 28dddcb51778..204d71b5f48c 100644
--- a/include/tools/XmlWalker.hxx
+++ b/include/tools/XmlWalker.hxx
@@ -11,9 +11,11 @@
 #define INCLUDED_TOOLS_XMLWALKER_HXX
 
 #include <tools/toolsdllapi.h>
-#include <tools/stream.hxx>
+#include <rtl/string.hxx>
 #include <memory>
 
+class SvStream;
+
 namespace tools
 {
 struct XmlWalkerImpl;
diff --git a/include/tools/XmlWriter.hxx b/include/tools/XmlWriter.hxx
index 6095d1379bfe..150bcffe0bf3 100644
--- a/include/tools/XmlWriter.hxx
+++ b/include/tools/XmlWriter.hxx
@@ -11,10 +11,12 @@
 #define INCLUDED_TOOLS_XMLWRITER_HXX
 
 #include <tools/toolsdllapi.h>
-#include <tools/stream.hxx>
+#include <rtl/ustring.hxx>
 #include <memory>
 #include <vector>
 
+class SvStream;
+
 namespace tools
 {
 struct XmlWriterImpl;
diff --git a/include/tools/b3dtrans.hxx b/include/tools/b3dtrans.hxx
index 181f43ca7bce..e42c7efbb76a 100644
--- a/include/tools/b3dtrans.hxx
+++ b/include/tools/b3dtrans.hxx
@@ -23,10 +23,10 @@
 #define ZBUFFER_DEPTH_RANGE         (double(256L * 256L * 256L))
 
 #include <basegfx/matrix/b3dhommatrix.hxx>
-#include <basegfx/range/b3drange.hxx>
 #include <tools/gen.hxx>
 #include <basegfx/matrix/b2dhommatrix.hxx>
-#include <basegfx/point/b2dpoint.hxx>
+#include <basegfx/point/b3dpoint.hxx>
+#include <basegfx/vector/b3dvector.hxx>
 #include <tools/toolsdllapi.h>
 
 /// Transformation sets for 3D output
diff --git a/include/tools/bigint.hxx b/include/tools/bigint.hxx
index 90fe56c8abce..66ce1dd3b1b8 100644
--- a/include/tools/bigint.hxx
+++ b/include/tools/bigint.hxx
@@ -19,17 +19,11 @@
 #ifndef INCLUDED_TOOLS_BIGINT_HXX
 #define INCLUDED_TOOLS_BIGINT_HXX
 
-#include <climits>
 #include <rtl/ustring.hxx>
 #include <tools/toolsdllapi.h>
-#include <tools/solar.h>
-
-class SvStream;
 
 #define MAX_DIGITS 8
 
-class Fraction;
-
 class SAL_WARN_UNUSED TOOLS_DLLPUBLIC BigInt
 {
 private:
diff --git a/include/tools/contnr.hxx b/include/tools/contnr.hxx
index 5b04ea7bc820..716cbf242bd1 100644
--- a/include/tools/contnr.hxx
+++ b/include/tools/contnr.hxx
@@ -19,8 +19,6 @@
 #ifndef INCLUDED_TOOLS_CONTNR_HXX
 #define INCLUDED_TOOLS_CONTNR_HXX
 
-#include <tools/toolsdllapi.h>
-
 #include <limits.h>
 
 #define TREELIST_APPEND             (ULONG_MAX)
diff --git a/include/tools/date.hxx b/include/tools/date.hxx
index 69d9e3d03ea3..03f0057c27c8 100644
--- a/include/tools/date.hxx
+++ b/include/tools/date.hxx
@@ -19,14 +19,13 @@
 #ifndef INCLUDED_TOOLS_DATE_HXX
 #define INCLUDED_TOOLS_DATE_HXX
 
-#include <sal/log.hxx>
-
 #include <tools/toolsdllapi.h>
 
 #include <ostream>
 
 #include <com/sun/star/util/Date.hpp>
-#include <com/sun/star/util/DateTime.hpp>
+
+namespace com { namespace sun { namespace star { namespace util { struct DateTime; } } } }
 
 enum DayOfWeek { MONDAY, TUESDAY, WEDNESDAY, THURSDAY, FRIDAY,
                  SATURDAY, SUNDAY };
diff --git a/include/tools/debug.hxx b/include/tools/debug.hxx
index 3fa2d5f9460a..b66a9b2866ef 100644
--- a/include/tools/debug.hxx
+++ b/include/tools/debug.hxx
@@ -22,7 +22,6 @@
 #include <tools/toolsdllapi.h>
 
 #include <sal/detail/log.h>
-#include <sal/types.h>
 
 /** The facilities provided by this header are deprecated.  True assertions
     (that detect broken program logic) should use standard assert (which aborts
diff --git a/include/tools/diagnose_ex.h b/include/tools/diagnose_ex.h
index 0f924fc20988..6d63c215dedc 100644
--- a/include/tools/diagnose_ex.h
+++ b/include/tools/diagnose_ex.h
@@ -27,7 +27,6 @@
 
 #include <tools/toolsdllapi.h>
 #include <cppuhelper/exc_hlp.hxx>
-#include <osl/thread.h>
 
 TOOLS_DLLPUBLIC void DbgUnhandledException(const css::uno::Any& caughtException,
         const char* currentFunction, const char* fileAndLineNo,
diff --git a/include/tools/gen.hxx b/include/tools/gen.hxx
index 90486536c401..5d6ff5ea66b8 100644
--- a/include/tools/gen.hxx
+++ b/include/tools/gen.hxx
@@ -24,7 +24,6 @@
 #include <limits.h>
 #include <algorithm>
 #include <ostream>
-#include <cstdlib>
 
 class SvStream;
 namespace rtl
diff --git a/include/tools/globname.hxx b/include/tools/globname.hxx
index 1a9a19e42fca..3280cb69a2f6 100644
--- a/include/tools/globname.hxx
+++ b/include/tools/globname.hxx
@@ -19,8 +19,6 @@
 #ifndef INCLUDED_TOOLS_GLOBNAME_HXX
 #define INCLUDED_TOOLS_GLOBNAME_HXX
 
-#include <vector>
-
 #include <tools/toolsdllapi.h>
 #include <com/sun/star/uno/Sequence.hxx>
 #include <o3tl/cow_wrapper.hxx>
diff --git a/include/tools/inetmime.hxx b/include/tools/inetmime.hxx
index 11208eae50ef..80820a2b722a 100644
--- a/include/tools/inetmime.hxx
+++ b/include/tools/inetmime.hxx
@@ -22,7 +22,6 @@
 #include <tools/toolsdllapi.h>
 #include <rtl/character.hxx>
 #include <rtl/string.hxx>
-#include <rtl/strbuf.hxx>
 #include <rtl/ustring.hxx>
 #include <tools/debug.hxx>
 
diff --git a/include/tools/inetmsg.hxx b/include/tools/inetmsg.hxx
index 445c3a60922c..ed0e074fb167 100644
--- a/include/tools/inetmsg.hxx
+++ b/include/tools/inetmsg.hxx
@@ -21,7 +21,6 @@
 
 #include <tools/toolsdllapi.h>
 #include <rtl/string.hxx>
-#include <rtl/textenc.h>
 #include <rtl/ustring.hxx>
 #include <tools/inetmime.hxx>
 #include <tools/stream.hxx>
diff --git a/include/tools/inetstrm.hxx b/include/tools/inetstrm.hxx
index 765474ef8248..8c74b9703d08 100644
--- a/include/tools/inetstrm.hxx
+++ b/include/tools/inetstrm.hxx
@@ -26,7 +26,6 @@
 #include <memory>
 
 class INetMIMEMessage;
-class SvStream;
 
 class TOOLS_DLLPUBLIC INetMIMEMessageStream
 {
diff --git a/include/tools/poly.hxx b/include/tools/poly.hxx
index 1a0926ecfff3..17503b02bb09 100644
--- a/include/tools/poly.hxx
+++ b/include/tools/poly.hxx
@@ -21,7 +21,6 @@
 
 #include <tools/toolsdllapi.h>
 #include <tools/gen.hxx>
-#include <tools/debug.hxx>
 #include <o3tl/typed_flags_set.hxx>
 #include <o3tl/cow_wrapper.hxx>
 
@@ -61,7 +60,6 @@ enum class PolyFlags : sal_uInt8
 class SvStream;
 class ImplPolygon;
 struct ImplPolyPolygon;
-namespace tools { class PolyPolygon; }
 
 namespace basegfx
 {
diff --git a/include/tools/stream.hxx b/include/tools/stream.hxx
index f6dc99155a1c..ccd9d349540b 100644
--- a/include/tools/stream.hxx
+++ b/include/tools/stream.hxx
@@ -19,7 +19,6 @@
 #ifndef INCLUDED_TOOLS_STREAM_HXX
 #define INCLUDED_TOOLS_STREAM_HXX
 
-#include <limits>
 #include <tools/toolsdllapi.h>
 #include <tools/lineend.hxx>
 #include <tools/ref.hxx>
diff --git a/include/tools/svborder.hxx b/include/tools/svborder.hxx
index 856efecfdbe7..61e1270e1ccd 100644
--- a/include/tools/svborder.hxx
+++ b/include/tools/svborder.hxx
@@ -21,7 +21,8 @@
 #define INCLUDED_TOOLS_SVBORDER_HXX
 
 #include <tools/toolsdllapi.h>
-#include <tools/gen.hxx>
+
+namespace tools { class Rectangle; }
 
 class SAL_WARN_UNUSED TOOLS_DLLPUBLIC SvBorder
 {
diff --git a/include/tools/time.hxx b/include/tools/time.hxx
index 9c137e5b9196..bada8bd5a2d0 100644
--- a/include/tools/time.hxx
+++ b/include/tools/time.hxx
@@ -20,9 +20,9 @@
 #define INCLUDED_TOOLS_TIME_HXX
 
 #include <tools/toolsdllapi.h>
-#include <tools/solar.h>
 #include <com/sun/star/util/Time.hpp>
-#include <com/sun/star/util/DateTime.hpp>
+
+namespace com { namespace sun { namespace star { namespace util { struct DateTime; } } } }
 
 /**
  @WARNING: This class can serve both as wall clock time and time duration, and
diff --git a/include/tools/urlobj.hxx b/include/tools/urlobj.hxx
index 94e22af5a159..84556aa4a609 100644
--- a/include/tools/urlobj.hxx
+++ b/include/tools/urlobj.hxx
@@ -20,8 +20,6 @@
 #define INCLUDED_TOOLS_URLOBJ_HXX
 
 #include <tools/toolsdllapi.h>
-#include <com/sun/star/uno/Reference.hxx>
-#include <rtl/string.h>
 #include <rtl/ustrbuf.hxx>
 #include <rtl/textenc.h>
 #include <sal/types.h>
@@ -36,6 +34,8 @@ namespace com { namespace sun { namespace star { namespace util {
     class XStringWidth;
 } } } }
 
+namespace com { namespace sun { namespace star { namespace uno { template <typename > class Reference; } } } }
+
 // Common URL prefixes for various schemes:
 #define INET_FTP_SCHEME "ftp://"
 #define INET_HTTP_SCHEME "http://"
diff --git a/include/tools/weakbase.h b/include/tools/weakbase.h
index 82c16773e33c..458381e47fff 100644
--- a/include/tools/weakbase.h
+++ b/include/tools/weakbase.h
@@ -20,7 +20,6 @@
 #define INCLUDED_TOOLS_WEAKBASE_H
 
 #include <sal/types.h>
-#include <osl/diagnose.h>
 #include <rtl/ref.hxx>
 #include <tools/toolsdllapi.h>