[Libreoffice-commits] core.git: binaryurp/IwyuFilter_binaryurp.yaml binaryurp/source

Gabor Kelemen (via logerrit) logerrit at kemper.freedesktop.org
Mon Jan 27 12:55:11 UTC 2020


 binaryurp/IwyuFilter_binaryurp.yaml  |   31 +++++++++++++++++++++++++++++++
 binaryurp/source/bridge.cxx          |    3 ---
 binaryurp/source/bridge.hxx          |    1 -
 binaryurp/source/bridgefactory.cxx   |    1 -
 binaryurp/source/bridgefactory.hxx   |    3 ---
 binaryurp/source/currentcontext.cxx  |    3 ---
 binaryurp/source/incomingrequest.cxx |    1 -
 binaryurp/source/marshal.cxx         |    2 --
 binaryurp/source/unmarshal.cxx       |    3 ---
 9 files changed, 31 insertions(+), 17 deletions(-)

New commits:
commit 5405681347b0e6881ca78d92517a066df47a3366
Author:     Gabor Kelemen <kelemen.gabor2 at nisz.hu>
AuthorDate: Sat Jan 11 03:13:08 2020 +0100
Commit:     Miklos Vajna <vmiklos at collabora.com>
CommitDate: Mon Jan 27 13:54:16 2020 +0100

    tdf#42949 Fix IWYU warnings in binaryurp/
    
    Found with bin/find-unneeded-includes
    Only removal proposals are dealt with here.
    
    Change-Id: I293036c7f6ab1af9b9b8736558c6d37124ec6d21
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87116
    Tested-by: Jenkins
    Reviewed-by: Miklos Vajna <vmiklos at collabora.com>

diff --git a/binaryurp/IwyuFilter_binaryurp.yaml b/binaryurp/IwyuFilter_binaryurp.yaml
new file mode 100644
index 000000000000..5eb9515a45fc
--- /dev/null
+++ b/binaryurp/IwyuFilter_binaryurp.yaml
@@ -0,0 +1,31 @@
+---
+assumeFilename: binaryurp/source/bridge.cxx
+blacklist:
+    binaryurp/source/marshal.hxx:
+    # Don't propose hxx -> h change in URE libs
+    - rtl/byteseq.hxx
+    binaryurp/source/reader.hxx:
+    # Don't propose hxx -> h change in URE libs
+    - rtl/byteseq.hxx
+    binaryurp/source/incomingrequest.hxx:
+    # Don't propose hxx -> h change in URE libs
+    - rtl/byteseq.hxx
+    binaryurp/source/lessoperators.cxx:
+    # Don't propose hxx -> h change in URE libs
+    - rtl/byteseq.hxx
+    binaryurp/source/outgoingrequests.cxx:
+    # Don't propose hxx -> h change in URE libs
+    - rtl/byteseq.hxx
+    # Actually needed
+    - lessoperators.hxx
+    binaryurp/source/bridgefactory.cxx:
+    # Actually needed
+    - com/sun/star/connection/XConnection.hpp
+    - com/sun/star/uno/XComponentContext.hpp
+    - com/sun/star/uno/XInterface.hpp
+    binaryurp/source/bridge.cxx:
+    # Actually needed
+    - com/sun/star/bridge/XInstanceProvider.hpp
+    - com/sun/star/connection/XConnection.hpp
+    - com/sun/star/lang/XEventListener.hpp
+    - com/sun/star/uno/XInterface.hpp
diff --git a/binaryurp/source/bridge.cxx b/binaryurp/source/bridge.cxx
index 8b086b71119b..5e2b5e6bc874 100644
--- a/binaryurp/source/bridge.cxx
+++ b/binaryurp/source/bridge.cxx
@@ -46,12 +46,9 @@
 #include <rtl/byteseq.hxx>
 #include <rtl/random.h>
 #include <rtl/ref.hxx>
-#include <rtl/ustrbuf.hxx>
-#include <rtl/ustring.h>
 #include <rtl/ustring.hxx>
 #include <sal/log.hxx>
 #include <sal/types.h>
-#include <typelib/typeclass.h>
 #include <typelib/typedescription.h>
 #include <typelib/typedescription.hxx>
 #include <uno/dispatcher.hxx>
diff --git a/binaryurp/source/bridge.hxx b/binaryurp/source/bridge.hxx
index 4572e7e573c2..d6528a3f115b 100644
--- a/binaryurp/source/bridge.hxx
+++ b/binaryurp/source/bridge.hxx
@@ -29,7 +29,6 @@
 #include <com/sun/star/bridge/XBridge.hpp>
 #include <com/sun/star/lang/XComponent.hpp>
 #include <com/sun/star/uno/Reference.hxx>
-#include <com/sun/star/uno/RuntimeException.hpp>
 #include <cppuhelper/implbase.hxx>
 #include <osl/conditn.hxx>
 #include <osl/mutex.hxx>
diff --git a/binaryurp/source/bridgefactory.cxx b/binaryurp/source/bridgefactory.cxx
index 6563b101f1df..350e9a77df26 100644
--- a/binaryurp/source/bridgefactory.cxx
+++ b/binaryurp/source/bridgefactory.cxx
@@ -21,7 +21,6 @@
 
 #include <algorithm>
 #include <cassert>
-#include <exception>
 
 #include <com/sun/star/bridge/BridgeExistsException.hpp>
 #include <com/sun/star/connection/XConnection.hpp>
diff --git a/binaryurp/source/bridgefactory.hxx b/binaryurp/source/bridgefactory.hxx
index 1a83962ee6b7..5e49610eeb42 100644
--- a/binaryurp/source/bridgefactory.hxx
+++ b/binaryurp/source/bridgefactory.hxx
@@ -22,15 +22,12 @@
 
 #include <sal/config.h>
 
-#include <exception>
 #include <vector>
 #include <map>
 
 #include <com/sun/star/bridge/XBridgeFactory2.hpp>
 #include <com/sun/star/lang/XServiceInfo.hpp>
-#include <com/sun/star/uno/Exception.hpp>
 #include <com/sun/star/uno/Reference.hxx>
-#include <com/sun/star/uno/RuntimeException.hpp>
 #include <cppuhelper/basemutex.hxx>
 #include <cppuhelper/compbase.hxx>
 #include <sal/types.h>
diff --git a/binaryurp/source/currentcontext.cxx b/binaryurp/source/currentcontext.cxx
index 56c3306aae29..acaf606d269c 100644
--- a/binaryurp/source/currentcontext.cxx
+++ b/binaryurp/source/currentcontext.cxx
@@ -19,10 +19,7 @@
 
 #include <sal/config.h>
 
-#include <com/sun/star/uno/Reference.hxx>
 #include <com/sun/star/uno/RuntimeException.hpp>
-#include <com/sun/star/uno/XInterface.hpp>
-#include <rtl/ustring.h>
 #include <rtl/ustring.hxx>
 #include <uno/current_context.h>
 #include <uno/dispatcher.hxx>
diff --git a/binaryurp/source/incomingrequest.cxx b/binaryurp/source/incomingrequest.cxx
index 95e85a3f5808..eed6f7cb8466 100644
--- a/binaryurp/source/incomingrequest.cxx
+++ b/binaryurp/source/incomingrequest.cxx
@@ -20,7 +20,6 @@
 #include <sal/config.h>
 
 #include <cassert>
-#include <list>
 #include <vector>
 
 #include <com/sun/star/bridge/XInstanceProvider.hpp>
diff --git a/binaryurp/source/marshal.cxx b/binaryurp/source/marshal.cxx
index 7003759e3d14..7d60cbf4de8a 100644
--- a/binaryurp/source/marshal.cxx
+++ b/binaryurp/source/marshal.cxx
@@ -22,10 +22,8 @@
 #include <cassert>
 #include <vector>
 
-#include <com/sun/star/uno/Reference.hxx>
 #include <com/sun/star/uno/RuntimeException.hpp>
 #include <com/sun/star/uno/Sequence.hxx>
-#include <com/sun/star/uno/XInterface.hpp>
 #include <cppu/unotype.hxx>
 #include <rtl/byteseq.hxx>
 #include <rtl/string.hxx>
diff --git a/binaryurp/source/unmarshal.cxx b/binaryurp/source/unmarshal.cxx
index 41145ccffca5..f317c9c59b7d 100644
--- a/binaryurp/source/unmarshal.cxx
+++ b/binaryurp/source/unmarshal.cxx
@@ -22,13 +22,10 @@
 #include <cassert>
 #include <cstdlib>
 #include <new>
-#include <vector>
 
 #include <com/sun/star/io/IOException.hpp>
-#include <com/sun/star/uno/Reference.hxx>
 #include <com/sun/star/uno/RuntimeException.hpp>
 #include <com/sun/star/uno/Sequence.hxx>
-#include <com/sun/star/uno/XInterface.hpp>
 #include <cppu/unotype.hxx>
 #include <rtl/byteseq.hxx>
 #include <rtl/ref.hxx>


More information about the Libreoffice-commits mailing list