diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2017-12-16 21:21:37 +0200 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2018-03-19 18:58:54 +0100 |
| commit | d0f8d9e82925231f4921425a6d55ad18bd0e73ac (patch) | |
| tree | a9238b4fb3964430216e3627da59818dfca58da6 /cui | |
| parent | b2ebb68ddebf584a08933db5d2351e47b0055c99 (diff) | |
loplugin:unusedindex Remove useless outer loop
Change-Id: I6e5e7162b1e0ea2b9f33866ea76e2614acdffb06
Reviewed-on: https://gerrit.libreoffice.org/46606
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
(cherry picked from commit ccb6a1c6d59859467f9d9a6d954501e9329ee202)
Diffstat (limited to 'cui')
| -rw-r--r-- | cui/source/tabpages/numpages.cxx | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/cui/source/tabpages/numpages.cxx b/cui/source/tabpages/numpages.cxx index a6e69c4d023b..c8cded8ce25d 100644 --- a/cui/source/tabpages/numpages.cxx +++ b/cui/source/tabpages/numpages.cxx @@ -1718,19 +1718,16 @@ IMPL_LINK_NOARG(SvxNumOptionsTabPage, PreviewInvalidateHdl_Impl, Timer *, void) IMPL_LINK( SvxNumOptionsTabPage, AllLevelHdl_Impl, Edit&, rBox, void ) { - for(sal_uInt16 i = 0; i < pActNum->GetLevelCount(); i++) + sal_uInt16 nMask = 1; + for(sal_uInt16 e = 0; e < pActNum->GetLevelCount(); e++) { - sal_uInt16 nMask = 1; - for(sal_uInt16 e = 0; e < pActNum->GetLevelCount(); e++) + if(nActNumLvl & nMask) { - if(nActNumLvl & nMask) - { - SvxNumberFormat aNumFmt(pActNum->GetLevel(e)); - aNumFmt.SetIncludeUpperLevels((sal_uInt8) std::min(static_cast<NumericField&>(rBox).GetValue(), sal_Int64(e + 1)) ); - pActNum->SetLevel(e, aNumFmt); - } - nMask <<= 1; + SvxNumberFormat aNumFmt(pActNum->GetLevel(e)); + aNumFmt.SetIncludeUpperLevels((sal_uInt8) std::min(static_cast<NumericField&>(rBox).GetValue(), sal_Int64(e + 1)) ); + pActNum->SetLevel(e, aNumFmt); } + nMask <<= 1; } SetModified(); } |