diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-07 08:42:54 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-07 10:45:05 +0200 |
commit | 868d8c8f0fdf376b0a3eb545ee841c9c12ffee3b (patch) | |
tree | 7e8b919732f3d73cfb77974c489ee864103b2882 /sc/source/core/tool/token.cxx | |
parent | 9479171a09ba4c73afa8b40a5c2590df3b6d5415 (diff) |
loplugin:unnecessaryparen handle parens inside call expr
stick to single-arg function calls, sometimes parens in multi-arg calls
might be there for clarity
Change-Id: Ib80190c571ce65b5d219a88056687042de749e74
Reviewed-on: https://gerrit.libreoffice.org/39676
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/core/tool/token.cxx')
-rw-r--r-- | sc/source/core/tool/token.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sc/source/core/tool/token.cxx b/sc/source/core/tool/token.cxx index 5b1e193f6738..46a33554f318 100644 --- a/sc/source/core/tool/token.cxx +++ b/sc/source/core/tool/token.cxx @@ -5109,9 +5109,9 @@ namespace { void wrapAddress( ScAddress& rPos, SCCOL nMaxCol, SCROW nMaxRow ) { if (rPos.Col() > nMaxCol) - rPos.SetCol((rPos.Col() % (nMaxCol+1))); + rPos.SetCol(rPos.Col() % (nMaxCol+1)); if (rPos.Row() > nMaxRow) - rPos.SetRow((rPos.Row() % (nMaxRow+1))); + rPos.SetRow(rPos.Row() % (nMaxRow+1)); } template<typename T> void wrapRange( T& n1, T& n2, T nMax ) |