VNCConn: remove unused getFrameBufferRegion(const wxRect& rect, wxBitmap& dst)

re #152

Author: bk138

src/VNCConn.cpp

@@ -1805,58 +1805,6 @@ wxBitmap VNCConn::getFrameBufferRegion(const wxRect& rect)
 }
 
 
-
-
-bool VNCConn::getFrameBufferRegion(const wxRect& rect, wxBitmap& dst)
-{
-  wxCriticalSectionLocker lock(mutex_framebuffer);
-
-  // sanity check requested region against framebuffer
-  if(rect.x < 0 || rect.x + rect.width > getFrameBufferWidth()
-     || rect.y < 0 || rect.y + rect.height > getFrameBufferHeight())
-       return false;
-
-  // check dst against framebuffer
-  if(dst.GetWidth() != getFrameBufferWidth() || dst.GetHeight() != getFrameBufferHeight())
-    return false;
-
-  /*
-    copy directly from framebuffer into the destination bitmap
-  */
-  wxAlphaPixelData dst_data(dst);
-  wxAlphaPixelData::Iterator dst_it(dst_data);
-  dst_it.Offset(dst_data, rect.x, rect.y);
-
-  int bytesPerPixel = cl->format.bitsPerPixel/8;
-  uint8_t *fbsub_it = cl->frameBuffer + rect.y*cl->width*bytesPerPixel + rect.x*bytesPerPixel;
-
-  for( int y = 0; y < rect.height; ++y )
-    {
-      wxAlphaPixelData::Iterator dst_it_rowStart = dst_it;
-      uint8_t *fbsub_it_rowStart = fbsub_it;
-
-      for( int x = 0; x < rect.width; ++x, ++dst_it, fbsub_it += bytesPerPixel)
-	{
-	  dst_it.Red() = *(fbsub_it+0);	  
-	  dst_it.Green() = *(fbsub_it+1);	  
-	  dst_it.Blue() = *(fbsub_it+2);	  
-	  dst_it.Alpha() = 255; // 255 is opaque, libvncclient always sets this byte to 0
-	}
-      
-      // CR
-      dst_it = dst_it_rowStart;
-      fbsub_it = fbsub_it_rowStart;
-      
-      // LF
-      dst_it.OffsetY(dst_data, 1);
-      fbsub_it += cl->width * bytesPerPixel;
-    }
-
-  return true;
-}
-
-
-
 int VNCConn::getFrameBufferWidth()
 {
   wxCriticalSectionLocker lock(mutex_framebuffer);

src/VNCConn.h

@@ -167,8 +167,6 @@ class VNCConn: public wxEvtHandler, public wxThreadHelper
 
   // returns a wxBitmap (this uses COW, so is okay)
   wxBitmap getFrameBufferRegion(const wxRect& region);
-  // writes requested region directly into dst bitmap which must have the same dimensions as the framebuffer
-  bool getFrameBufferRegion(const wxRect& rect, wxBitmap& dst);
   int getFrameBufferWidth();
   int getFrameBufferHeight();
   int getFrameBufferDepth();