[Libreoffice-commits] core.git: codemaker/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Wed Jul 25 17:28:29 UTC 2018


 codemaker/source/cppumaker/dependencies.hxx |    3 +++
 codemaker/source/cppumaker/includes.cxx     |    2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 082c7caecef2658920998b0877683f513c35db34
Author:     Muhammet Kara <muhammet.kara at pardus.org.tr>
AuthorDate: Wed Jul 25 17:00:20 2018 +0300
Commit:     Muhammet Kara <muhammet.kara at pardus.org.tr>
CommitDate: Wed Jul 25 19:28:03 2018 +0200

    Add missing method hasUnsignedLongDependency()
    
    m_unsignedLongDependency was already set but never used.
    
    Behavior of Includes::Includes ctor is now in consistency with
    the related switch cases in Includes::add (see: m_includeSalTypesH)
    
    Change-Id: I4cca1bab014ac280b73b7532aadfd5ff2b0b9894
    Reviewed-on: https://gerrit.libreoffice.org/57792
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>
    Tested-by: Jenkins

diff --git a/codemaker/source/cppumaker/dependencies.hxx b/codemaker/source/cppumaker/dependencies.hxx
index 718dac392c87..c7eae1dfc6b6 100644
--- a/codemaker/source/cppumaker/dependencies.hxx
+++ b/codemaker/source/cppumaker/dependencies.hxx
@@ -82,6 +82,9 @@ public:
 
     bool hasLongDependency() const { return m_longDependency; }
 
+    bool hasUnsignedLongDependency() const
+    { return m_unsignedLongDependency; }
+
     bool hasHyperDependency() const { return m_hyperDependency; }
 
     bool hasUnsignedHyperDependency() const
diff --git a/codemaker/source/cppumaker/includes.cxx b/codemaker/source/cppumaker/includes.cxx
index ddd541640671..53ca16afcd4d 100644
--- a/codemaker/source/cppumaker/includes.cxx
+++ b/codemaker/source/cppumaker/includes.cxx
@@ -57,7 +57,7 @@ Includes::Includes(
         || dependencies.hasShortDependency()
         || dependencies.hasUnsignedShortDependency()
         || dependencies.hasLongDependency()
-        || dependencies.hasUnsignedShortDependency()
+        || dependencies.hasUnsignedLongDependency()
         || dependencies.hasHyperDependency()
         || dependencies.hasUnsignedHyperDependency()
         || dependencies.hasCharDependency()),


More information about the Libreoffice-commits mailing list