diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2021-09-08 12:38:39 +0100 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-09-10 14:18:09 +0200 |
| commit | 53ebbad48c8c2a71044917a5cbac9657126e5adc (patch) | |
| tree | 32ae305d00d765b2f0d1bbb7f9bc4cefcac8cd18 /svtools | |
| parent | e32d864dbe086d630a8b17c2d376e320aee0253a (diff) | |
warning: Excessive padding in 'struct OutDevState'
experiment with optin.performance.Padding analyzer and tuning
AllowedPad to 2
with clang as compiler;
export ENVCFLAGSCXX="-Xclang -analyze -Xclang -analyzer-checker=optin.performance -Xclang -analyzer-config -Xclang optin.performance.Padding:AllowedPad=2"
and get a report of:
OutDevState (13 padding bytes, where 5 is optimal)
UIControlOptions (9 padding bytes, where 1 is optimal)
Change-Id: I174f6f62d6e32df057d5e0f21d4274166f6ed8c8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121812
Tested-by: Jenkins
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svtools')
| -rw-r--r-- | svtools/source/brwbox/datwin.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svtools/source/brwbox/datwin.cxx b/svtools/source/brwbox/datwin.cxx index a7bbde76fe28..87ae8ef1d92c 100644 --- a/svtools/source/brwbox/datwin.cxx +++ b/svtools/source/brwbox/datwin.cxx @@ -575,8 +575,8 @@ BrowseEvent::BrowseEvent( vcl::Window* pWindow, sal_Int32 nAbsRow, sal_uInt16 nColumn, sal_uInt16 nColumnId, const tools::Rectangle& rRect ): pWin(pWindow), - nRow(nAbsRow), aRect(rRect), + nRow(nAbsRow), nCol(nColumn), nColId(nColumnId) { |