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 /codemaker | |
| 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 'codemaker')
| -rw-r--r-- | codemaker/source/cppumaker/cppuoptions.cxx | 2 | ||||
| -rw-r--r-- | codemaker/source/javamaker/javaoptions.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/codemaker/source/cppumaker/cppuoptions.cxx b/codemaker/source/cppumaker/cppuoptions.cxx index 2a15f3497283..ea15907cc1e5 100644 --- a/codemaker/source/cppumaker/cppuoptions.cxx +++ b/codemaker/source/cppumaker/cppuoptions.cxx @@ -43,7 +43,7 @@ bool CppuOptions::initOptions(int ac, char* av[], bool bCmdFile) OString name(av[0]); sal_Int32 index = name.lastIndexOf(SEPARATOR); - m_program = name.copy((index > 0 ? index+1 : 0)); + m_program = name.copy(index > 0 ? index+1 : 0); if (ac < 2) { diff --git a/codemaker/source/javamaker/javaoptions.cxx b/codemaker/source/javamaker/javaoptions.cxx index cffc74451ac7..a09107a99b53 100644 --- a/codemaker/source/javamaker/javaoptions.cxx +++ b/codemaker/source/javamaker/javaoptions.cxx @@ -41,7 +41,7 @@ bool JavaOptions::initOptions(int ac, char* av[], bool bCmdFile) OString name(av[0]); sal_Int32 index = name.lastIndexOf(SEPARATOR); - m_program = name.copy((index > 0 ? index+1 : 0)); + m_program = name.copy(index > 0 ? index+1 : 0); if (ac < 2) { |