diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-04-14 15:44:22 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-04-14 20:38:31 +0200 |
commit | 9693b491295336955f7ce8359284b67a82ecf28f (patch) | |
tree | 0338cd2a61173c4c91dae5c85578e73a650f8702 /extensions/source/scanner/grid.cxx | |
parent | 6e44d68b2beba30efc48bdaf3e8f00e236bf53f2 (diff) |
loplugin:flatten in extensions
Change-Id: I58b5a2c12e464e568c4f66d398bbf0b1d8afc154
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92195
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'extensions/source/scanner/grid.cxx')
-rw-r--r-- | extensions/source/scanner/grid.cxx | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/extensions/source/scanner/grid.cxx b/extensions/source/scanner/grid.cxx index 89ea65898d01..4560d8f2baf7 100644 --- a/extensions/source/scanner/grid.cxx +++ b/extensions/source/scanner/grid.cxx @@ -277,23 +277,23 @@ double GridWindow::findMaxY() void GridWindow::computeExtremes() { - if( m_nValues && m_pXValues && m_pOrigYValues ) + if( !(m_nValues && m_pXValues && m_pOrigYValues) ) + return; + + m_fMaxX = m_fMinX = m_pXValues[0]; + m_fMaxY = m_fMinY = m_pOrigYValues[0]; + for( int i = 1; i < m_nValues; i++ ) { - m_fMaxX = m_fMinX = m_pXValues[0]; - m_fMaxY = m_fMinY = m_pOrigYValues[0]; - for( int i = 1; i < m_nValues; i++ ) - { - if( m_pXValues[ i ] > m_fMaxX ) - m_fMaxX = m_pXValues[ i ]; - else if( m_pXValues[ i ] < m_fMinX ) - m_fMinX = m_pXValues[ i ]; - if( m_pOrigYValues[ i ] > m_fMaxY ) - m_fMaxY = m_pOrigYValues[ i ]; - else if( m_pOrigYValues[ i ] < m_fMinY ) - m_fMinY = m_pOrigYValues[ i ]; - } - setBoundings( m_fMinX, m_fMinY, m_fMaxX, m_fMaxY ); + if( m_pXValues[ i ] > m_fMaxX ) + m_fMaxX = m_pXValues[ i ]; + else if( m_pXValues[ i ] < m_fMinX ) + m_fMinX = m_pXValues[ i ]; + if( m_pOrigYValues[ i ] > m_fMaxY ) + m_fMaxY = m_pOrigYValues[ i ]; + else if( m_pOrigYValues[ i ] < m_fMinY ) + m_fMinY = m_pOrigYValues[ i ]; } + setBoundings( m_fMinX, m_fMinY, m_fMaxX, m_fMaxY ); } |