]> source.dussan.org Git - tigervnc.git/commitdiff
Avoid translating simple error exceptions
authorPierre Ossman <ossman@cendio.se>
Sat, 17 Dec 2016 13:04:39 +0000 (14:04 +0100)
committerPierre Ossman <ossman@cendio.se>
Tue, 3 Jan 2017 12:50:32 +0000 (13:50 +0100)
These are very rare and very low level. Might as well just print
the offending call to keep things easily maintained.

vncviewer/OSXPixelBuffer.cxx
vncviewer/Win32PixelBuffer.cxx
vncviewer/X11PixelBuffer.cxx

index e9100f270d223bc7288a113ad5128a368f85e57a..d04d9b303e6bdada1acc3ff88891395132fdb0bc 100644 (file)
@@ -49,13 +49,13 @@ OSXPixelBuffer::OSXPixelBuffer(int width, int height) :
 
   lut = CGColorSpaceCreateDeviceRGB();
   if (!lut)
-    throw rfb::Exception(_("Could not create framebuffer device"));
+    throw rfb::Exception("CGColorSpaceCreateDeviceRGB");
 
   bitmap = CGBitmapContextCreate(data, width, height, 8, width*4, lut,
                                  kCGImageAlphaNoneSkipFirst | kCGBitmapByteOrder32Little);
   CGColorSpaceRelease(lut);
   if (!bitmap)
-    throw rfb::Exception(_("Could not create framebuffer bitmap"));
+    throw rfb::Exception("CGBitmapContextCreate");
 }
 
 
index 3f8253041a82cdfac17b6a8184f6ee3abc4271d8..002d4c82c6c3cf135ee2018289e64bbf23656b2c 100644 (file)
@@ -57,10 +57,8 @@ Win32PixelBuffer::Win32PixelBuffer(int width, int height) :
 
   bitmap = CreateDIBSection(NULL, (BITMAPINFO*)&bih,
                             DIB_RGB_COLORS, (void**)&data, NULL, 0);
-  if (!bitmap) {
-    int err = GetLastError();
-    throw rdr::SystemException(_("unable to create DIB section"), err);
-  }
+  if (!bitmap)
+    throw rdr::SystemException("CreateDIBSection", GetLastError());
 }
 
 
@@ -76,10 +74,10 @@ void Win32PixelBuffer::draw(int src_x, int src_y, int x, int y, int w, int h)
 
   dc = CreateCompatibleDC(fl_gc);
   if (!dc)
-    throw rdr::SystemException(_("CreateCompatibleDC failed"), GetLastError());
+    throw rdr::SystemException("CreateCompatibleDC", GetLastError());
 
   if (!SelectObject(dc, bitmap))
-    throw rdr::SystemException(_("SelectObject failed"), GetLastError());
+    throw rdr::SystemException("SelectObject", GetLastError());
 
   if (!BitBlt(fl_gc, x, y, w, h, dc, src_x, src_y, SRCCOPY)) {
     // If the desktop we're rendering to is inactive (like when the screen
@@ -88,7 +86,7 @@ void Win32PixelBuffer::draw(int src_x, int src_y, int x, int y, int w, int h)
     // with it. For now, we've only seen this error and for this function
     // so only ignore this combination.
     if (GetLastError() != ERROR_INVALID_HANDLE)
-      throw rdr::SystemException(_("BitBlt failed"), GetLastError());
+      throw rdr::SystemException("BitBlt", GetLastError());
   }
 
   DeleteDC(dc);
index ce5c4d82afe3c8957b369592ec0d552b3f26b601..a1d7687ed681119493527af556480aa180a84bd8 100644 (file)
@@ -110,11 +110,13 @@ X11PixelBuffer::X11PixelBuffer(int width, int height) :
     xim = XCreateImage(fl_display, fl_visual->visual, fl_visual->depth,
                        ZPixmap, 0, 0, width, height, BitmapPad(fl_display), 0);
     if (!xim)
-      throw rfb::Exception(_("Could not create framebuffer image"));
+      throw rfb::Exception("XCreateImage");
 
     xim->data = (char*)malloc(xim->bytes_per_line * xim->height);
     if (!xim->data)
       throw rfb::Exception(_("Not enough memory for framebuffer"));
+
+    vlog.debug("Using standard XImage");
   }
 
   data = (rdr::U8*)xim->data;