]> git.saurik.com Git - wxWidgets.git/blobdiff - contrib/src/fl/antiflickpl.cpp
Don't create uneeded variables.
[wxWidgets.git] / contrib / src / fl / antiflickpl.cpp
index 12ce48a1a7847c2a470b708bf6e8182b774158d6..5b153650402fc03d0129ac6bbb9f3a2171f874c6 100644 (file)
@@ -6,13 +6,9 @@
 // Created:     23/10/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Aleksandras Gluchovas
-// Licence:    wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
  
-#ifdef __GNUG__
-    #pragma implementation "antiflickpl.h"
-#endif
-
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
@@ -32,8 +28,8 @@ IMPLEMENT_DYNAMIC_CLASS( cbAntiflickerPlugin, cbPluginBase )
 
 BEGIN_EVENT_TABLE( cbAntiflickerPlugin, cbPluginBase )
 
-       EVT_PL_START_DRAW_IN_AREA  ( cbAntiflickerPlugin::OnStartDrawInArea  )
-       EVT_PL_FINISH_DRAW_IN_AREA ( cbAntiflickerPlugin::OnFinishDrawInArea )
+    EVT_PL_START_DRAW_IN_AREA  ( cbAntiflickerPlugin::OnStartDrawInArea  )
+    EVT_PL_FINISH_DRAW_IN_AREA ( cbAntiflickerPlugin::OnFinishDrawInArea )
 
 END_EVENT_TABLE()
 
@@ -49,188 +45,185 @@ wxMemoryDC* cbAntiflickerPlugin::mpHorizBufDc = 0;
 // constructors
 
 cbAntiflickerPlugin::cbAntiflickerPlugin(void)
-       : mpLRUBufDc  ( NULL ),
-         mLRUArea    ( -1,-1, -1,-1 )
+    : mpLRUBufDc  ( NULL ),
+      mLRUArea    ( -1,-1, -1,-1 )
 {
-       ++mRefCount;
+    ++mRefCount;
 }
 
 cbAntiflickerPlugin::cbAntiflickerPlugin( wxFrameLayout* pPanel, int paneMask )
 
-       : cbPluginBase( pPanel, paneMask ),
-         mpLRUBufDc  ( NULL ),
-         mLRUArea    ( -1,-1, -1,-1 )
+    : cbPluginBase( pPanel, paneMask ),
+      mpLRUBufDc  ( NULL ),
+      mLRUArea    ( -1,-1, -1,-1 )
 {
-       ++mRefCount;
+    ++mRefCount;
 }
 
 cbAntiflickerPlugin::~cbAntiflickerPlugin()
 {
-       if ( --mRefCount == 0 )
-       {
-               if ( mpHorizBuf )
-               {
-                       mpHorizBufDc->SelectObject( wxNullBitmap );
-                       delete mpHorizBuf;
-                       delete mpHorizBufDc;
-                       mpHorizBuf   = 0;
-                       mpHorizBufDc = 0;
-               }
-
-               if ( mpVertBuf ) 
-               {
-                       mpVertBufDc->SelectObject( wxNullBitmap );
-                       delete mpVertBuf; 
-                       delete mpVertBufDc;
-                       mpVertBuf   = 0;
-                       mpVertBufDc = 0;
-               }
-       }
+    if ( --mRefCount == 0 )
+    {
+        if ( mpHorizBuf )
+        {
+            mpHorizBufDc->SelectObject( wxNullBitmap );
+            delete mpHorizBuf;
+            delete mpHorizBufDc;
+            mpHorizBuf   = 0;
+            mpHorizBufDc = 0;
+        }
+
+        if ( mpVertBuf ) 
+        {
+            mpVertBufDc->SelectObject( wxNullBitmap );
+            delete mpVertBuf; 
+            delete mpVertBufDc;
+            mpVertBuf   = 0;
+            mpVertBufDc = 0;
+        }
+    }
 }
 
 wxDC* cbAntiflickerPlugin::FindSuitableBuffer( const wxRect& forArea )
 {
-       if ( mpVertBuf )
-       {
-               if ( mpVertBuf->GetHeight() >= forArea.height &&
-                        mpVertBuf->GetWidth()  >= forArea.width )
-
-                        return mpVertBufDc;
-       }
-       else
-       if ( mpHorizBuf )
-       {
-               if ( mpHorizBuf->GetHeight() >= forArea.height &&
-                        mpHorizBuf->GetWidth()  >= forArea.width )
-
-                        return mpHorizBufDc;
-       }
-
-       return 0;
+    if ( mpVertBuf )
+    {
+        if ( mpVertBuf->GetHeight() >= forArea.height &&
+             mpVertBuf->GetWidth()  >= forArea.width )
+            return mpVertBufDc;
+    }
+    else
+        if ( mpHorizBuf )
+        {
+            if ( mpHorizBuf->GetHeight() >= forArea.height &&
+                 mpHorizBuf->GetWidth()  >= forArea.width )
+                return mpHorizBufDc;
+        }
+
+    return 0;
 }
 
 wxDC* cbAntiflickerPlugin::AllocNewBuffer( const wxRect& forArea )
 {
-       // TBD:: preallocate bit larger bitmap at once, to avoid
-       //       excessive realocations later
-
-       // check whether the given area is oriented horizontally
-       // or vertically and choose corresponding bitmap to create or
-       // recreate
-
-       if ( forArea.height > forArea.width )
-       {
-               wxSize prevDim( 0,0 );
-
-               if ( mpVertBuf ) 
-               {
-                       prevDim.x = mpVertBuf->GetWidth();
-                       prevDim.y = mpVertBuf->GetHeight();
-
-                       mpVertBufDc->SelectObject( wxNullBitmap );
-                       delete mpVertBuf;
-               }
-               else
-                       mpVertBufDc = new wxMemoryDC();
-               
-               mpVertBuf = new wxBitmap( int( wxMax(forArea.width,  prevDim.x ) ), 
-                                                                 int( wxMax(forArea.height, prevDim.y ) )
-                                                               );
-
-               mpVertBufDc->SelectObject( *mpVertBuf );
-
-               return mpVertBufDc;
-       }
-       else
-       {
-               wxSize prevDim( 0,0 );
-
-               if ( mpHorizBuf ) 
-               {
-                       prevDim.x = mpHorizBuf->GetWidth();
-                       prevDim.y = mpHorizBuf->GetHeight();
-
-                       mpHorizBufDc->SelectObject( wxNullBitmap );
-                       delete mpHorizBuf;
-               }
-               else
-                       mpHorizBufDc = new wxMemoryDC();
-               
-               mpHorizBuf = new wxBitmap( int( wxMax(forArea.width,  prevDim.x ) ), 
-                                                                  int( wxMax(forArea.height, prevDim.y ) )
-                                                                );
-
-               mpHorizBufDc->SelectObject( *mpHorizBuf );
-
-               return mpHorizBufDc;
-       }
+    // TBD:: preallocate bit larger bitmap at once, to avoid
+    //       excessive realocations later
+
+    // check whether the given area is oriented horizontally
+    // or vertically and choose corresponding bitmap to create or
+    // recreate
+
+    if ( forArea.height > forArea.width )
+    {
+        wxSize prevDim( 0,0 );
+
+        if ( mpVertBuf ) 
+        {
+            prevDim.x = mpVertBuf->GetWidth();
+            prevDim.y = mpVertBuf->GetHeight();
+
+            mpVertBufDc->SelectObject( wxNullBitmap );
+            delete mpVertBuf;
+        }
+        else
+            mpVertBufDc = new wxMemoryDC();
+
+        mpVertBuf = new wxBitmap( int( wxMax(forArea.width,  prevDim.x ) ), 
+                                  int( wxMax(forArea.height, prevDim.y ) )
+                                );
+
+        mpVertBufDc->SelectObject( *mpVertBuf );
+
+        return mpVertBufDc;
+    }
+    else
+    {
+        wxSize prevDim( 0,0 );
+
+        if ( mpHorizBuf ) 
+        {
+            prevDim.x = mpHorizBuf->GetWidth();
+            prevDim.y = mpHorizBuf->GetHeight();
+
+            mpHorizBufDc->SelectObject( wxNullBitmap );
+            delete mpHorizBuf;
+        }
+        else
+            mpHorizBufDc = new wxMemoryDC();
+
+        mpHorizBuf = new wxBitmap( int( wxMax(forArea.width,  prevDim.x ) ), 
+                                   int( wxMax(forArea.height, prevDim.y ) )
+                                 );
+
+        mpHorizBufDc->SelectObject( *mpHorizBuf );
+
+        return mpHorizBufDc;
+    }
 }
 
 void cbAntiflickerPlugin::OnStartDrawInArea( cbStartDrawInAreaEvent& event )
 {
-       wxASSERT( mpLRUBufDc == NULL ); // DBG:: see comments in OnFinishDrawInArea(..) method
+    wxASSERT( mpLRUBufDc == NULL ); // DBG:: see comments in OnFinishDrawInArea(..) method
 
-       // short-cut
-       wxRect& area = event.mArea;
+    // short-cut
+    wxRect& area = event.mArea;
 
-       if ( event.mArea.width  < 0 || 
-                event.mArea.height < 0 ) return;
+    if ( event.mArea.width  < 0 || 
+         event.mArea.height < 0 ) return;
 
-       // memorize given area
-       mLRUArea.x      = area.x;
-       mLRUArea.y      = area.y;
-       mLRUArea.width  = area.width;
-       mLRUArea.height = area.height;
+    // memorize given area
+    mLRUArea.x      = area.x;
+    mLRUArea.y      = area.y;
+    mLRUArea.width  = area.width;
+    mLRUArea.height = area.height;
 
-       wxDC* pBufDc = FindSuitableBuffer( area );
+    wxDC* pBufDc = FindSuitableBuffer( area );
 
-       if ( !pBufDc )
-       
-               pBufDc = AllocNewBuffer( area );
+    if ( !pBufDc )
+        pBufDc = AllocNewBuffer( area );
 
-       pBufDc->SetDeviceOrigin( -area.x, -area.y );
+    pBufDc->SetDeviceOrigin( -area.x, -area.y );
 
-       pBufDc->SetClippingRegion( area.x,     area.y,
-                                                          area.width, area.height );
+    pBufDc->SetClippingRegion( area.x,     area.y,
+                               area.width, area.height );
 
-       wxClientDC clntDc( &mpLayout->GetParentFrame() );
+    wxClientDC clntDc( &mpLayout->GetParentFrame() );
 
-       (*event.mppDc) = pBufDc;
+    (*event.mppDc) = pBufDc;
 
-       mpLRUBufDc = pBufDc; // memorize buffer, which will be flushed to screen
-                            // upon "commiting" the drawing
+    mpLRUBufDc = pBufDc; // memorize buffer, which will be flushed to screen
+                         // upon "commiting" the drawing
 
-       /*
-       // OLD STUFF::
-       mpLRUBufDc->Blit( pos.x, pos.y, size.x, size.y,
-                                     &clntDc, pos.x, pos.y, wxCOPY );
-                                       */
+    /*
+    // OLD STUFF::
+    mpLRUBufDc->Blit( pos.x, pos.y, size.x, size.y,
+                      &clntDc, pos.x, pos.y, wxCOPY );
+                    */
 }
 
 void cbAntiflickerPlugin::OnFinishDrawInArea( cbFinishDrawInAreaEvent& event )
 {
-       wxRect& area = event.mArea;
+    wxRect& area = event.mArea;
 
-       if ( event.mArea.width  < 0 || 
-                event.mArea.height < 0 ) return;
+    if ( event.mArea.width  < 0 || 
+         event.mArea.height < 0 ) return;
 
-       wxASSERT( mpLRUBufDc ); // DBG:: OnStartDrawInArea should be called first
+    wxASSERT( mpLRUBufDc ); // DBG:: OnStartDrawInArea should be called first
 
-       // FOR NOW:: OnStartDrawInArea(..) should be immediately followed
-       //           by OnFinishDrawInArea(..) for the same area
+    // FOR NOW:: OnStartDrawInArea(..) should be immediately followed
+    //           by OnFinishDrawInArea(..) for the same area
 
-       wxASSERT( mLRUArea.x      == area.x      );
-       wxASSERT( mLRUArea.y      == area.y      );
-       wxASSERT( mLRUArea.width  == area.width  );
-       wxASSERT( mLRUArea.height == area.height );
+    wxASSERT( mLRUArea.x      == area.x      );
+    wxASSERT( mLRUArea.y      == area.y      );
+    wxASSERT( mLRUArea.width  == area.width  );
+    wxASSERT( mLRUArea.height == area.height );
 
-       wxClientDC clntDc( &mpLayout->GetParentFrame() );
+    wxClientDC clntDc( &mpLayout->GetParentFrame() );
 
-       // "commit" drawings in one-shot
-       clntDc.Blit( area.x, area.y, area.width, area.height,
-                                mpLRUBufDc, area.x, area.y, wxCOPY );
+    // "commit" drawings in one-shot
+    clntDc.Blit( area.x, area.y, area.width, area.height,
+                 mpLRUBufDc, area.x, area.y, wxCOPY );
 
-       mpLRUBufDc->DestroyClippingRegion();
-       mpLRUBufDc = 0;
+    mpLRUBufDc->DestroyClippingRegion();
+    mpLRUBufDc = 0;
 }