[Libreoffice-commits] core.git: Branch 'feature/gsoc-calc-enhanced-db-range' - 2 commits - sc/inc sc/source

Kohei Yoshida kohei.yoshida at gmail.com
Mon Aug 19 14:51:38 PDT 2013


 sc/inc/dbdata.hxx               |    4 ++--
 sc/source/core/tool/dbdata2.cxx |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit ee6a3b678319b0740a338781cf6cce6e1bd963e0
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date:   Mon Aug 19 17:52:18 2013 -0400

    Ensure correct initialization order.
    
    Change-Id: Ife8a4f0654d1ca6e9342750ff53c6f3e00bbc286

diff --git a/sc/inc/dbdata.hxx b/sc/inc/dbdata.hxx
index f337a38..a7a49b5 100644
--- a/sc/inc/dbdata.hxx
+++ b/sc/inc/dbdata.hxx
@@ -58,7 +58,6 @@ private:
     bool            bDoSize;
     bool            bKeepFmt;
     bool            bStripData;
-    bool            bHasFormatting;
 
     /// QueryParam
     bool            bIsAdvanced;        ///< true if created by advanced filter
@@ -69,6 +68,7 @@ private:
     sal_uInt16      nIndex;             ///< unique index formulas
     bool            bAutoFilter;        ///< AutoFilter? (not saved)
     bool            bModified;          ///< is set/cleared for/by(?) UpdateReference
+    bool            bHasFormatting;
 
     using ScRefreshTimer::operator==;
 
commit 82acea96bbf9ac89feddbf91e1881330b26c253b
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date:   Mon Aug 19 17:48:16 2013 -0400

    Don't return a reference to a temporary variable.
    
    That's a recipe for trouble.
    
    Change-Id: I78647225209e57cf88c04718d99a07142983369d

diff --git a/sc/inc/dbdata.hxx b/sc/inc/dbdata.hxx
index e6a35ed..f337a38 100644
--- a/sc/inc/dbdata.hxx
+++ b/sc/inc/dbdata.hxx
@@ -156,7 +156,7 @@ public:
                         SCsCOL nDx, SCsROW nDy, SCsTAB nDz);
 
     void ExtendDataArea(ScDocument* pDoc);
-    const OUString& GetCellStyle( const ScAddress& rPos, bool bRowStripe );
+    OUString GetCellStyle( const ScAddress& rPos, bool bRowStripe );
 };
 
 class SC_DLLPUBLIC ScDBCollection
diff --git a/sc/source/core/tool/dbdata2.cxx b/sc/source/core/tool/dbdata2.cxx
index 0f70a6f..879437a 100644
--- a/sc/source/core/tool/dbdata2.cxx
+++ b/sc/source/core/tool/dbdata2.cxx
@@ -11,7 +11,7 @@
 #include "dbdata.hxx"
 #include "dbdataformatting.hxx"
 
-const OUString& ScDBData::GetCellStyle( const ScAddress& rPos, bool bRowStripe )
+OUString ScDBData::GetCellStyle( const ScAddress& rPos, bool bRowStripe )
 {
     //first see if the DB Range has any DB Formatting at all
     if( !HasFormatting() )


More information about the Libreoffice-commits mailing list