summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJorenz Paragas <j.paragas.237@gmail.com>2016-04-25 23:41:12 -0700
committerMichael Stahl <mstahl@redhat.com>2016-04-26 11:53:52 +0000
commit6a4a15c87c03feffb90cc416ce22d2819e41442d (patch)
tree3a580524af4fbbbd43ad0f3d0586d654d424f2bf
parentc3572701cf5481edc7432b062e0fa69697c212be (diff)
tdf#42949: clean up includes in include/oox/crypto with iwyu
The includes in CryptTools.hxx that are within #if/#endif directives are not altered. iwyu suggested replacing the includes for nss.h, pk11pub.h, and sechash.h with hasht.h, pkcs11t.h, seccomon.h, and secmodt.h. I decided not to because it might make CryptTools.hxx harder to refactor in exchange for little gain. Before creating this commit, I ran 'make check' after passing --with-tls=openssl to ./autogen.sh to ensure that building with either OpenSSL or NSS (the default except on iOS and Android) works. Change-Id: I20260d18f073ffd8077bbcc597e7a8e4954e2ec5 Reviewed-on: https://gerrit.libreoffice.org/24386 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
-rw-r--r--include/oox/crypto/AgileEngine.hxx9
-rw-r--r--include/oox/crypto/CryptTools.hxx5
-rw-r--r--include/oox/crypto/CryptoEngine.hxx9
-rw-r--r--include/oox/crypto/DocumentDecryption.hxx24
-rw-r--r--include/oox/crypto/DocumentEncryption.hxx13
-rw-r--r--include/oox/crypto/Standard2007Engine.hxx8
-rw-r--r--oox/source/core/xmlfilterbase.cxx1
-rw-r--r--oox/source/crypto/AgileEngine.cxx3
-rw-r--r--oox/source/crypto/DocumentDecryption.cxx6
-rw-r--r--oox/source/crypto/DocumentEncryption.cxx9
-rw-r--r--oox/source/crypto/Standard2007Engine.cxx4
11 files changed, 64 insertions, 27 deletions
diff --git a/include/oox/crypto/AgileEngine.hxx b/include/oox/crypto/AgileEngine.hxx
index cf47947f1e97..2bb73305d441 100644
--- a/include/oox/crypto/AgileEngine.hxx
+++ b/include/oox/crypto/AgileEngine.hxx
@@ -11,8 +11,17 @@
#ifndef INCLUDED_OOX_CRYPTO_AGILEENGINE_HXX
#define INCLUDED_OOX_CRYPTO_AGILEENGINE_HXX
+#include <vector>
+
#include <oox/crypto/CryptTools.hxx>
#include <oox/crypto/CryptoEngine.hxx>
+#include <rtl/ustring.hxx>
+#include <sal/types.h>
+
+namespace oox {
+ class BinaryXInputStream;
+ class BinaryXOutputStream;
+}
namespace oox {
namespace core {
diff --git a/include/oox/crypto/CryptTools.hxx b/include/oox/crypto/CryptTools.hxx
index d93fef62e336..23e48c29d86b 100644
--- a/include/oox/crypto/CryptTools.hxx
+++ b/include/oox/crypto/CryptTools.hxx
@@ -22,8 +22,6 @@
#include <config_oox.h>
-#include <rtl/ustring.hxx>
-
#if USE_TLS_OPENSSL
#include <openssl/evp.h>
#include <openssl/sha.h>
@@ -35,9 +33,10 @@
#include <sechash.h>
#endif // USE_TLS_NSS
-#include <rtl/digest.h>
#include <vector>
+#include <sal/types.h>
+
namespace oox {
namespace core {
diff --git a/include/oox/crypto/CryptoEngine.hxx b/include/oox/crypto/CryptoEngine.hxx
index a6ad858fb5db..9b988c0b2f25 100644
--- a/include/oox/crypto/CryptoEngine.hxx
+++ b/include/oox/crypto/CryptoEngine.hxx
@@ -13,8 +13,13 @@
#include <vector>
-#include <oox/helper/binaryinputstream.hxx>
-#include <oox/helper/binaryoutputstream.hxx>
+#include <rtl/ustring.hxx>
+#include <sal/types.h>
+
+namespace oox {
+ class BinaryXInputStream;
+ class BinaryXOutputStream;
+}
namespace oox {
namespace core {
diff --git a/include/oox/crypto/DocumentDecryption.hxx b/include/oox/crypto/DocumentDecryption.hxx
index d6b867b47b29..16029ad6998b 100644
--- a/include/oox/crypto/DocumentDecryption.hxx
+++ b/include/oox/crypto/DocumentDecryption.hxx
@@ -13,20 +13,22 @@
#include <oox/dllapi.h>
-#include <oox/ole/olestorage.hxx>
-#include <oox/helper/binaryinputstream.hxx>
-#include <oox/helper/binaryoutputstream.hxx>
+#include <memory>
-#include <com/sun/star/io/XStream.hpp>
-#include <com/sun/star/beans/NamedValue.hpp>
-#include <com/sun/star/uno/Sequence.hxx>
+#include <com/sun/star/uno/Reference.h>
+#include <com/sun/star/uno/Sequence.h>
+#include <oox/crypto/CryptoEngine.hxx>
+#include <rtl/ustring.hxx>
-#include <oox/crypto/CryptTools.hxx>
-#include <oox/crypto/AgileEngine.hxx>
-#include <oox/crypto/Standard2007Engine.hxx>
+namespace com { namespace sun { namespace star {
+ namespace beans { struct NamedValue; }
+ namespace io { class XInputStream; }
+ namespace io { class XStream; }
+ namespace uno { class XComponentContext; }
+} } }
-#include <memory>
-#include <vector>
+namespace oox { class BinaryInputStream; }
+namespace oox { namespace ole { class OleStorage; } }
namespace oox {
namespace core {
diff --git a/include/oox/crypto/DocumentEncryption.hxx b/include/oox/crypto/DocumentEncryption.hxx
index 720cb38b9e14..42646f367d1a 100644
--- a/include/oox/crypto/DocumentEncryption.hxx
+++ b/include/oox/crypto/DocumentEncryption.hxx
@@ -13,16 +13,15 @@
#include <oox/dllapi.h>
-#include <oox/ole/olestorage.hxx>
-
-#include <com/sun/star/io/XStream.hpp>
-#include <com/sun/star/beans/NamedValue.hpp>
-
-#include <oox/crypto/CryptTools.hxx>
+#include <com/sun/star/uno/Reference.h>
#include <oox/crypto/Standard2007Engine.hxx>
+#include <rtl/ustring.hxx>
-#include <vector>
+namespace com { namespace sun { namespace star {
+ namespace io { class XStream; }
+} } }
+namespace oox { namespace ole { class OleStorage; } }
namespace oox {
namespace core {
diff --git a/include/oox/crypto/Standard2007Engine.hxx b/include/oox/crypto/Standard2007Engine.hxx
index a4d13fb0f2cb..05bdc995e1a8 100644
--- a/include/oox/crypto/Standard2007Engine.hxx
+++ b/include/oox/crypto/Standard2007Engine.hxx
@@ -11,8 +11,14 @@
#ifndef INCLUDED_OOX_CRYPTO_STANDARD2007ENGINE_HXX
#define INCLUDED_OOX_CRYPTO_STANDARD2007ENGINE_HXX
-#include <oox/crypto/CryptTools.hxx>
#include <oox/crypto/CryptoEngine.hxx>
+#include <rtl/ustring.hxx>
+#include <sal/types.h>
+
+namespace oox {
+ class BinaryXInputStream;
+ class BinaryXOutputStream;
+}
namespace oox {
namespace core {
diff --git a/oox/source/core/xmlfilterbase.cxx b/oox/source/core/xmlfilterbase.cxx
index a3cd57606e8e..066a3d6432c6 100644
--- a/oox/source/core/xmlfilterbase.cxx
+++ b/oox/source/core/xmlfilterbase.cxx
@@ -44,6 +44,7 @@
#include "oox/helper/containerhelper.hxx"
#include "oox/helper/propertyset.hxx"
#include "oox/helper/zipstorage.hxx"
+#include <oox/ole/olestorage.hxx>
#include <oox/token/namespaces.hxx>
#include "oox/token/properties.hxx"
#include <oox/token/tokens.hxx>
diff --git a/oox/source/crypto/AgileEngine.cxx b/oox/source/crypto/AgileEngine.cxx
index 13780868cd55..3897b877b294 100644
--- a/oox/source/crypto/AgileEngine.cxx
+++ b/oox/source/crypto/AgileEngine.cxx
@@ -10,6 +10,9 @@
#include "oox/crypto/AgileEngine.hxx"
+#include <oox/helper/binaryinputstream.hxx>
+#include <oox/helper/binaryoutputstream.hxx>
+
namespace oox {
namespace core {
diff --git a/oox/source/crypto/DocumentDecryption.cxx b/oox/source/crypto/DocumentDecryption.cxx
index a57b5671063d..eea0c629d641 100644
--- a/oox/source/crypto/DocumentDecryption.cxx
+++ b/oox/source/crypto/DocumentDecryption.cxx
@@ -15,11 +15,17 @@
#include <cppuhelper/implbase.hxx>
#include <com/sun/star/io/XSeekable.hpp>
+#include <com/sun/star/io/XStream.hpp>
#include <com/sun/star/uno/XComponentContext.hpp>
#include <com/sun/star/xml/sax/XFastParser.hpp>
#include <com/sun/star/xml/sax/XFastTokenHandler.hpp>
#include <com/sun/star/xml/sax/FastParser.hpp>
#include <com/sun/star/xml/sax/FastToken.hpp>
+#include <oox/crypto/AgileEngine.hxx>
+#include <oox/crypto/Standard2007Engine.hxx>
+#include <oox/helper/binaryinputstream.hxx>
+#include <oox/helper/binaryoutputstream.hxx>
+#include <oox/ole/olestorage.hxx>
namespace oox {
namespace core {
diff --git a/oox/source/crypto/DocumentEncryption.cxx b/oox/source/crypto/DocumentEncryption.cxx
index cd6527aa95a2..1c328fccd775 100644
--- a/oox/source/crypto/DocumentEncryption.cxx
+++ b/oox/source/crypto/DocumentEncryption.cxx
@@ -10,15 +10,18 @@
#include "oox/crypto/DocumentEncryption.hxx"
+#include <com/sun/star/io/XInputStream.hpp>
+#include <com/sun/star/io/XOutputStream.hpp>
+#include <com/sun/star/io/XStream.hpp>
#include <com/sun/star/io/XSeekable.hpp>
-#include "oox/helper/binaryinputstream.hxx"
-#include "oox/helper/binaryoutputstream.hxx"
+#include <oox/helper/binaryinputstream.hxx>
+#include <oox/helper/binaryoutputstream.hxx>
+#include <oox/ole/olestorage.hxx>
namespace oox {
namespace core {
-using namespace css::beans;
using namespace css::io;
using namespace css::lang;
using namespace css::uno;
diff --git a/oox/source/crypto/Standard2007Engine.cxx b/oox/source/crypto/Standard2007Engine.cxx
index 0280a174b1a1..34fa2265a719 100644
--- a/oox/source/crypto/Standard2007Engine.cxx
+++ b/oox/source/crypto/Standard2007Engine.cxx
@@ -10,7 +10,11 @@
#include "oox/crypto/Standard2007Engine.hxx"
+#include <oox/crypto/CryptTools.hxx>
+#include <oox/helper/binaryinputstream.hxx>
+#include <oox/helper/binaryoutputstream.hxx>
#include <osl/time.h>
+#include <rtl/digest.h>
#include <rtl/random.h>
namespace oox {