From c39fd46c07a4c9a3a521a1fc6a111f380a42431c Mon Sep 17 00:00:00 2001 From: Maciej Suminski Date: Fri, 9 Feb 2018 14:20:11 +0100 Subject: [PATCH] PROGRESS_REPORTER: code formatting --- common/widgets/progress_reporter.cpp | 27 ++++++++++++++++++--------- include/widgets/progress_reporter.h | 10 +++++----- 2 files changed, 23 insertions(+), 14 deletions(-) diff --git a/common/widgets/progress_reporter.cpp b/common/widgets/progress_reporter.cpp index d405a90e6a..53068bf7c8 100644 --- a/common/widgets/progress_reporter.cpp +++ b/common/widgets/progress_reporter.cpp @@ -32,7 +32,8 @@ PROGRESS_REPORTER::PROGRESS_REPORTER( int aNumPhases ) : m_progress( 0 ), m_maxProgress( 1 ) { -}; +} + void PROGRESS_REPORTER::BeginPhase( int aPhase ) { @@ -41,6 +42,7 @@ void PROGRESS_REPORTER::BeginPhase( int aPhase ) updateUI(); } + void PROGRESS_REPORTER::AdvancePhase( ) { m_phase++; @@ -48,31 +50,36 @@ void PROGRESS_REPORTER::AdvancePhase( ) updateUI(); } + void PROGRESS_REPORTER::Report( const wxString& aMessage ) { - m_rptMessage = aMessage; + m_rptMessage = aMessage; updateUI(); } -void PROGRESS_REPORTER::SetMaxProgress ( int aMaxProgress ) + +void PROGRESS_REPORTER::SetMaxProgress( int aMaxProgress ) { m_maxProgress = aMaxProgress; updateUI(); } -void PROGRESS_REPORTER::AdvanceProgress( ) + +void PROGRESS_REPORTER::AdvanceProgress() { m_progress++; } + int PROGRESS_REPORTER::currentProgress() const { - double current = ( 1.0 / (double)m_numPhases ) * + double current = ( 1.0 / (double) m_numPhases ) * ( (double) m_phase + ( (double) m_progress.load() / (double) m_maxProgress ) ); - return (int)(current * 1000); + return (int)( current * 1000 ); } + // Please, *DO NOT* use wxPD_APP_MODAL style: it is not necessary // (without this option the PROGRESS_REPORTER is modal for the parent frame) // and PROGRESS_REPORTER works fine on OSX only without this style @@ -103,14 +110,16 @@ void WX_PROGRESS_REPORTER::updateUI() wxProgressDialog::Update( cur, m_rptMessage ); } + void PROGRESS_REPORTER::KeepRefreshing( bool aWait ) { #ifdef USE_OPENMP - while ( m_progress < m_maxProgress && m_maxProgress > 0) + while( m_progress < m_maxProgress && m_maxProgress > 0 ) { updateUI(); - wxMilliSleep(10); - if ( !aWait ) + wxMilliSleep( 10 ); + + if( !aWait ) return; } #else diff --git a/include/widgets/progress_reporter.h b/include/widgets/progress_reporter.h index 34764b0c5e..5504b5c792 100644 --- a/include/widgets/progress_reporter.h +++ b/include/widgets/progress_reporter.h @@ -45,23 +45,23 @@ class PROGRESS_REPORTER /** * Uses the next vailable virtual zone of the dialog progress bar */ - void AdvancePhase( ); + void AdvancePhase(); /** * Display aMessage in the progress bar dialog */ - void Report ( const wxString& aMessage ); + void Report( const wxString& aMessage ); /** * Fix the value thar gives the 100 precent progress bar length * (inside the current virtual zone) */ - void SetMaxProgress ( int aMaxProgress ); + void SetMaxProgress( int aMaxProgress ); /** * Increment the progress bar length (inside the current virtual zone) */ - void AdvanceProgress( ); + void AdvanceProgress(); /** * Update the UI dialog. @@ -99,7 +99,7 @@ public: * aNumPhases = n creates n virtual progress bar zones: a 0 to 100 percent width * of a virtual zone fills 0 to 1/n progress bar full size of the nth virtual zone index */ - WX_PROGRESS_REPORTER( wxWindow *aParent, const wxString &aTitle, int aNumPhases ); + WX_PROGRESS_REPORTER( wxWindow* aParent, const wxString& aTitle, int aNumPhases ); ~WX_PROGRESS_REPORTER(); private: