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/vcl/print.hxx | |
| 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 'include/vcl/print.hxx')
| -rw-r--r-- | include/vcl/print.hxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/vcl/print.hxx b/include/vcl/print.hxx index dba696eebf18..e487e8aa82b5 100644 --- a/include/vcl/print.hxx +++ b/include/vcl/print.hxx @@ -533,13 +533,13 @@ public: struct UIControlOptions { OUString maDependsOnName; + OUString maGroupHint; + std::vector< css::beans::PropertyValue > + maAddProps; sal_Int32 mnDependsOnEntry; bool mbAttachToDependency; - OUString maGroupHint; bool mbInternalOnly; bool mbEnabled; - std::vector< css::beans::PropertyValue > - maAddProps; UIControlOptions( const OUString& i_rDependsOnName = OUString(), sal_Int32 i_nDependsOnEntry = -1, bool i_bAttachToDependency = false) |