[Libreoffice-commits] core.git: include/oox oox/source
Jorenz Paragas
j.paragas.237 at gmail.com
Tue Apr 26 11:54:16 UTC 2016
include/oox/crypto/AgileEngine.hxx | 9 +++++++++
include/oox/crypto/CryptTools.hxx | 5 ++---
include/oox/crypto/CryptoEngine.hxx | 9 +++++++--
include/oox/crypto/DocumentDecryption.hxx | 24 +++++++++++++-----------
include/oox/crypto/DocumentEncryption.hxx | 13 ++++++-------
include/oox/crypto/Standard2007Engine.hxx | 8 +++++++-
oox/source/core/xmlfilterbase.cxx | 1 +
oox/source/crypto/AgileEngine.cxx | 3 +++
oox/source/crypto/DocumentDecryption.cxx | 6 ++++++
oox/source/crypto/DocumentEncryption.cxx | 9 ++++++---
oox/source/crypto/Standard2007Engine.cxx | 4 ++++
11 files changed, 64 insertions(+), 27 deletions(-)
New commits:
commit 6a4a15c87c03feffb90cc416ce22d2819e41442d
Author: Jorenz Paragas <j.paragas.237 at gmail.com>
Date: Mon Apr 25 23:41:12 2016 -0700
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 at libreoffice.org>
Reviewed-by: Michael Stahl <mstahl at redhat.com>
diff --git a/include/oox/crypto/AgileEngine.hxx b/include/oox/crypto/AgileEngine.hxx
index cf47947..2bb7330 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 d93fef6..23e48c2 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 a6ad858..9b988c0 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 d6b867b..16029ad 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 720cb38..42646f3 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 a4d13fb0..05bdc99 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 a3cd576..066a3d6 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 1378086..3897b87 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 a57b567..eea0c62 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 cd6527a..1c328fc 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 0280a17..34fa226 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 {
More information about the Libreoffice-commits
mailing list