[Libreoffice-commits] core.git: sw/inc sw/source

Noel Grandin (via logerrit) logerrit at kemper.freedesktop.org
Sun Sep 12 15:28:00 UTC 2021


 sw/inc/viscrs.hxx              |    8 ++++----
 sw/source/core/crsr/callnk.hxx |    2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 96ce5bdf72fbcd4ca2907fc4ad71a038a11e2e22
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Sun Sep 12 13:16:34 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Sun Sep 12 17:27:26 2021 +0200

    clang:optin.performance.Padding in sw
    
    Excessive padding in 'class SwVisibleCursor' (12 padding bytes, where 4
    is optimal).
    Excessive padding in 'class SwCallLink' (10 padding bytes, where 2 is
    optimal).
    
    Change-Id: I8d23cc5f45ff5e916fa04205f4d92c4603514b1e
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121996
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sw/inc/viscrs.hxx b/sw/inc/viscrs.hxx
index 4818edfc4dfb..a14c2e207722 100644
--- a/sw/inc/viscrs.hxx
+++ b/sw/inc/viscrs.hxx
@@ -38,15 +38,15 @@ class SW_DLLPUBLIC SwVisibleCursor
     friend void InitCore();
     friend void FinitCore();
 
-    bool m_bIsVisible;
-    bool m_bIsDragCursor;
-
-    vcl::Cursor m_aTextCursor;
     const SwCursorShell* m_pCursorShell;
+    vcl::Cursor m_aTextCursor;
 
     /// For LibreOfficeKit only - remember what page we were at the last time.
     sal_uInt16 m_nPageLastTime;
 
+    bool m_bIsVisible;
+    bool m_bIsDragCursor;
+
 public:
     SwVisibleCursor( const SwCursorShell * pCShell );
     ~SwVisibleCursor();
diff --git a/sw/source/core/crsr/callnk.hxx b/sw/source/core/crsr/callnk.hxx
index ca47821c147f..e3875ddfa08b 100644
--- a/sw/source/core/crsr/callnk.hxx
+++ b/sw/source/core/crsr/callnk.hxx
@@ -33,9 +33,9 @@ class SwCallLink
 public:
     SwCursorShell & m_rShell;
     sal_uLong m_nNode;
+    tools::Long m_nLeftFramePos;
     sal_Int32 m_nContent;
     SwNodeType m_nNodeType;
-    tools::Long m_nLeftFramePos;
     bool m_bHasSelection;
 
     explicit SwCallLink( SwCursorShell & rSh );


More information about the Libreoffice-commits mailing list