commit 2d0dc3abee27c9383ad3ab23e02ecf94fcdd7596
author Pierre Ossman <ossman@cendio.se> Thu Apr 27 13:28:49 2017 +0200
committer Pierre Ossman <ossman@cendio.se> Fri Apr 28 11:45:42 2017 +0200
tree 3e3aea5b08e79f7ea5f2247c45f16dfd9768d78d
parent 0630434292b19898e6bd66e46cc4c6a641efdfe5

Use sub-image instead of clipping

It seems to be more efficient.

vncviewer/Surface_OSX.cxx

diff --git a/vncviewer/Surface_OSX.cxx b/vncviewer/Surface_OSX.cxx
index 2dfaa47..668187b 100644
--- a/vncviewer/Surface_OSX.cxx
+++ b/vncviewer/Surface_OSX.cxx
@@ -30,32 +30,36 @@
 
 static void render(CGContextRef gc, CGImageRef image,
                    CGBlendMode mode, CGFloat alpha,
-                   int src_x, int src_y, int src_w, int src_h,
+                   int src_x, int src_y,
                    int x, int y, int w, int h)
 {
   CGRect rect;
+  CGImageRef subimage;
+
+  rect.origin.x = src_x;
+  rect.origin.y = src_y;
+  rect.size.width = w;
+  rect.size.height = h;
+
+  subimage = CGImageCreateWithImageInRect(image, rect);
+  if (!subimage)
+    throw rdr::Exception("CGImageCreateImageWithImageInRect");
 
   CGContextSaveGState(gc);
 
   CGContextSetBlendMode(gc, mode);
   CGContextSetAlpha(gc, alpha);
 
-  // We have to use clipping to partially display an image
   rect.origin.x = x;
   rect.origin.y = y;
   rect.size.width = w;
   rect.size.height = h;
 
-  CGContextClipToRect(gc, rect);
-
-  rect.origin.x = x - src_x;
-  rect.origin.y = y - src_y;
-  rect.size.width = src_w;
-  rect.size.height = src_h;
-
-  CGContextDrawImage(gc, rect, image);
+  CGContextDrawImage(gc, rect, subimage);
 
   CGContextRestoreGState(gc);
+
+  CGImageRelease(subimage);
 }
 
 static CGContextRef make_bitmap(int width, int height, unsigned char* data)
@@ -108,11 +112,10 @@
   CGContextConcatCTM(fl_gc, CGAffineTransformInvert(CGContextGetCTM(fl_gc)));
 
   // macOS Coordinates are from bottom left, not top left
-  src_y = height() - (src_y + h);
   y = Fl_Window::current()->h() - (y + h);
 
   render(fl_gc, image, kCGBlendModeCopy, 1.0,
-         src_x, src_y, width(), height(), x, y, w, h);
+         src_x, src_y, x, y, w, h);
 
   CGContextRestoreGState(fl_gc);
 }
@@ -124,11 +127,10 @@
   bitmap = make_bitmap(dst->width(), dst->height(), dst->data);
 
   // macOS Coordinates are from bottom left, not top left
-  src_y = height() - (src_y + h);
   y = dst->height() - (y + h);
 
   render(bitmap, image, kCGBlendModeCopy, 1.0,
-         src_x, src_y, width(), height(), x, y, w, h);
+         src_x, src_y, x, y, w, h);
 
   CGContextRelease(bitmap);
 }
@@ -142,11 +144,10 @@
   CGContextConcatCTM(fl_gc, CGAffineTransformInvert(CGContextGetCTM(fl_gc)));
 
   // macOS Coordinates are from bottom left, not top left
-  src_y = height() - (src_y + h);
   y = Fl_Window::current()->h() - (y + h);
 
   render(fl_gc, image, kCGBlendModeNormal, (CGFloat)a/255.0,
-         src_x, src_y, width(), height(), x, y, w, h);
+         src_x, src_y, x, y, w, h);
 
   CGContextRestoreGState(fl_gc);
 }
@@ -158,11 +159,10 @@
   bitmap = make_bitmap(dst->width(), dst->height(), dst->data);
 
   // macOS Coordinates are from bottom left, not top left
-  src_y = height() - (src_y + h);
   y = dst->height() - (y + h);
 
   render(bitmap, image, kCGBlendModeNormal, (CGFloat)a/255.0,
-         src_x, src_y, width(), height(), x, y, w, h);
+         src_x, src_y, x, y, w, h);
 
   CGContextRelease(bitmap);
 }