Improve TQProgressBar repaint optimization

src/widgets/qprogressbar.cpp

@@ -212,13 +212,11 @@ void QProgressBar::setProgress( int progress )
 	 progress < 0 || ( ( progress > total_steps ) && total_steps ) )
 	return;
 
-    const bool needRepaint = isVisible() && requireRepaint( progress );
-
     progress_val = progress;
 
     setIndicator( progress_str, progress_val, total_steps );
 
-    if ( needRepaint ) {
+    if ( isVisible() && repaintRequired() ) {
         repaint( FALSE );
         d->last_painted_progress = progress;
     }
@@ -358,10 +356,9 @@ void QProgressBar::styleChange( QStyle& old )
     would require a repaint of the progress bar. This allows efficient
     repainting.
 */
-bool QProgressBar::requireRepaint( int newProgress ) const
+bool QProgressBar::repaintRequired() const
 {
-    if ( newProgress == progress_val ||
+    if ( progress_val == d->last_painted_progress ) {
-         newProgress == d->last_painted_progress ) {
         return false;
     }
 
@@ -375,7 +372,7 @@ bool QProgressBar::requireRepaint( int newProgress ) const
         progressPerPixel = float( total_steps ) / float( width );
     }
 
-    const int delta = d->last_painted_progress - newProgress;
+    const int delta = d->last_painted_progress - progress_val;
     return QABS( delta ) >= progressPerPixel;
 }
 

src/widgets/qprogressbar.h

@@ -96,7 +96,7 @@ protected:
     virtual bool setIndicator( QString & progress_str, int progress,
 			       int totalSteps );
     void styleChange( QStyle& );
-    bool	requireRepaint( int newProgress ) const;
+    bool	repaintRequired() const;
 
 private:
     int		total_steps;