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 /include/svtools | |
parent | tdf#144353 - Handling of missing optional parameters (diff) | |
download | core-53ebbad48c8c2a71044917a5cbac9657126e5adc.tar.gz core-53ebbad48c8c2a71044917a5cbac9657126e5adc.zip |
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 'include/svtools')
-rw-r--r-- | include/svtools/brwbox.hxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/svtools/brwbox.hxx b/include/svtools/brwbox.hxx index 1c7d6e5404cc..60e37de7d432 100644 --- a/include/svtools/brwbox.hxx +++ b/include/svtools/brwbox.hxx @@ -118,8 +118,8 @@ namespace o3tl class BrowseEvent { VclPtr<vcl::Window> pWin; - sal_Int32 nRow; tools::Rectangle aRect; + sal_Int32 nRow; sal_uInt16 nCol; sal_uInt16 nColId; |