diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-11-05 15:20:22 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-11-06 06:27:51 +0100 |
| commit | 7b99cdb2d77a3feaf9b782c1e656f2d922e2746e (patch) | |
| tree | 180459dcd8379ee1a2e89098a4d9eda72abe8f05 /basic | |
| parent | 7ee07296a66df29555c9e9a684f24bc68201cb78 (diff) | |
loplugin:indentation find broken if statements
so I don't read the "then" block as being a sequential statements
Change-Id: Ib2004acd3518bd4ebd2246f02a26c2c0a8bbab4c
Reviewed-on: https://gerrit.libreoffice.org/82069
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'basic')
| -rw-r--r-- | basic/source/basmgr/basmgr.cxx | 2 | ||||
| -rw-r--r-- | basic/source/comp/io.cxx | 22 | ||||
| -rw-r--r-- | basic/source/sbx/sbxconv.hxx | 9 |
3 files changed, 18 insertions, 15 deletions
diff --git a/basic/source/basmgr/basmgr.cxx b/basic/source/basmgr/basmgr.cxx index f0a7b38730a2..a2e59d8b4213 100644 --- a/basic/source/basmgr/basmgr.cxx +++ b/basic/source/basmgr/basmgr.cxx @@ -197,7 +197,7 @@ void BasMgrContainerListenerImpl::addLibraryModulesImpl( BasicManager const * pM pLib->MakeModule( aModuleName, aInfo, aMod ); } else - pLib->MakeModule( aModuleName, aMod ); + pLib->MakeModule( aModuleName, aMod ); } pLib->SetModified( false ); diff --git a/basic/source/comp/io.cxx b/basic/source/comp/io.cxx index d1abbee9c330..3a24d4141027 100644 --- a/basic/source/comp/io.cxx +++ b/basic/source/comp/io.cxx @@ -290,18 +290,18 @@ void SbiParser::Close() if( IsEoln( eCurTok ) ) aGen.Gen( SbiOpcode::CLOSE_, 0 ); else - for( ;; ) - { - SbiExpression aExpr( this ); - while( Peek() == COMMA || Peek() == SEMICOLON ) - Next(); - aExpr.Gen(); - aGen.Gen( SbiOpcode::CHANNEL_ ); - aGen.Gen( SbiOpcode::CLOSE_, 1 ); + for( ;; ) + { + SbiExpression aExpr( this ); + while( Peek() == COMMA || Peek() == SEMICOLON ) + Next(); + aExpr.Gen(); + aGen.Gen( SbiOpcode::CHANNEL_ ); + aGen.Gen( SbiOpcode::CLOSE_, 1 ); - if( IsEoln( Peek() ) ) - break; - } + if( IsEoln( Peek() ) ) + break; + } } diff --git a/basic/source/sbx/sbxconv.hxx b/basic/source/sbx/sbxconv.hxx index 8a8fd98a7a6d..378c752b88d9 100644 --- a/basic/source/sbx/sbxconv.hxx +++ b/basic/source/sbx/sbxconv.hxx @@ -70,9 +70,12 @@ sal_Int64 ImpGetCurrency( const SbxValues* ); void ImpPutCurrency( SbxValues*, const sal_Int64 ); inline sal_Int64 ImpDoubleToCurrency( double d ) - { if (d > 0) return static_cast<sal_Int64>( d * CURRENCY_FACTOR + 0.5); - else return static_cast<sal_Int64>( d * CURRENCY_FACTOR - 0.5); - } +{ + if (d > 0) + return static_cast<sal_Int64>( d * CURRENCY_FACTOR + 0.5); + else + return static_cast<sal_Int64>( d * CURRENCY_FACTOR - 0.5); +} inline double ImpCurrencyToDouble( const sal_Int64 r ) { return static_cast<double>(r) / double(CURRENCY_FACTOR); } |