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

Stephan Bergmann sbergman at redhat.com
Tue Jan 23 20:52:52 UTC 2018


 sc/source/core/data/pagepar.cxx |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit d1016591f42b47ea52d96ce9f09ed01749b97d77
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Jan 23 11:46:44 2018 +0100

    Use a more conventional way of clearing ScRange variables
    
    ...to avoid upcoming GCC 8 -Werror=class-memaccess ("clearing an object of type
    ‘class ScRange’ with no trivial copy-assignment")
    
    Change-Id: I2178697082b1c284d26cedaa64c73b92d9e6aecb
    Reviewed-on: https://gerrit.libreoffice.org/48393
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/sc/source/core/data/pagepar.cxx b/sc/source/core/data/pagepar.cxx
index 97d0d03eb54a..9853e4ac52a5 100644
--- a/sc/source/core/data/pagepar.cxx
+++ b/sc/source/core/data/pagepar.cxx
@@ -17,8 +17,6 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#include <string.h>
-
 #include <pagepar.hxx>
 
 // struct ScPageTableParam:
@@ -59,9 +57,9 @@ void ScPageAreaParam::Reset()
 {
     bPrintArea = bRepeatRow = bRepeatCol = false;
 
-    memset( &aPrintArea, 0, sizeof(ScRange) );
-    memset( &aRepeatRow, 0, sizeof(ScRange) );
-    memset( &aRepeatCol, 0, sizeof(ScRange) );
+    aPrintArea = ScRange();
+    aRepeatRow = ScRange();
+    aRepeatCol = ScRange();
 }
 
 


More information about the Libreoffice-commits mailing list