diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-09-11 15:51:39 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-09-11 17:24:02 +0200 |
| commit | d114a341bc9e891b2e10ff60ddadb0b5f58e6381 (patch) | |
| tree | b14182329cc38066253c192e1ea5ccc989f1a05c /sfx2 | |
| parent | 580b314a797917def3ce0f52abc02700a2768120 (diff) | |
clang:optin.performance.Padding in sfx2
Excessive padding in 'class SfxControllerItem' (10 padding bytes, where
2 is optimal).
Excessive padding in 'struct SfxChildWinInfo' (11 padding bytes, where 3
is optimal).
Excessive padding in 'struct SfxChildWinFactory' (12 padding bytes,
where 4 is optimal).
Excessive padding in 'class SfxChildWindow' (10 padding bytes, where 2
is optimal).
Change-Id: I435cfcf8a199bcab4612d671e1900ba659b8e383
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121941
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/appl/childwin.cxx | 4 | ||||
| -rw-r--r-- | sfx2/source/control/ctrlitem.cxx | 8 |
2 files changed, 6 insertions, 6 deletions
diff --git a/sfx2/source/appl/childwin.cxx b/sfx2/source/appl/childwin.cxx index 404d912d9d3e..b00f70a3afb3 100644 --- a/sfx2/source/appl/childwin.cxx +++ b/sfx2/source/appl/childwin.cxx @@ -143,9 +143,9 @@ bool GetSplitSizeFromString( const OUString& rStr, Size& rSize ) SfxChildWindow::SfxChildWindow(vcl::Window *pParentWindow, sal_uInt16 nId) : pParent(pParentWindow) - , nType(nId) - , eChildAlignment(SfxChildAlignment::NOALIGNMENT) , pImpl(new SfxChildWindow_Impl) + , eChildAlignment(SfxChildAlignment::NOALIGNMENT) + , nType(nId) { pImpl->pFact = nullptr; pImpl->bHideNotDelete = false; diff --git a/sfx2/source/control/ctrlitem.cxx b/sfx2/source/control/ctrlitem.cxx index b55ceda7d349..28edfec666de 100644 --- a/sfx2/source/control/ctrlitem.cxx +++ b/sfx2/source/control/ctrlitem.cxx @@ -170,19 +170,19 @@ void SfxControllerItem::SetId( sal_uInt16 nItemId ) // creates an atomic item for a controller without registration. SfxControllerItem::SfxControllerItem() - : nId(0) - , pNext(this) + : pNext(this) , pBindings(nullptr) , eFallbackCoreMetric(MapUnit::Map100thMM) + , nId(0) { } // creates a representation of the function nId and registers it SfxControllerItem::SfxControllerItem(sal_uInt16 nID, SfxBindings &rBindings) - : nId(nID) - , pNext(this) + : pNext(this) , pBindings(&rBindings) , eFallbackCoreMetric(MapUnit::Map100thMM) + , nId(nID) { Bind(nId, &rBindings); } |