More chromium_code whack-a-mole.
authorNico Weber <thakis@chromium.org>
Fri, 30 Oct 2015 20:07:00 +0000 (13:07 -0700)
committerNico Weber <thakis@chromium.org>
Fri, 30 Oct 2015 20:07:00 +0000 (13:07 -0700)
BUG=pdfium:29
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1419943006 .

BUILD.gn
core/src/fxge/win32/fx_win32_device.cpp
core/src/fxge/win32/fx_win32_gdipext.cpp
core/src/fxge/win32/fx_win32_print.cpp
pdfium.gyp

index 3e1b89c..120abc2 100644 (file)
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -40,8 +40,10 @@ config("pdfium_config") {
       "/wd4345",
       # TODO(thestig): Fix all instances, remove this, pdfium:29
       "/wd4245",
+      "/wd4310",
       "/wd4389",
       "/wd4701",
+      "/wd4702",
       "/wd4706",
       "/wd4800",
     ]
index 93c2fd6..a32f9ad 100644 (file)
@@ -358,7 +358,7 @@ void* CFX_Win32FontInfo::MapFont(int weight,
                     OUT_TT_ONLY_PRECIS, 0, 0, subst_pitch_family, face);
   char facebuf[100];
   HFONT hOldFont = (HFONT)::SelectObject(m_hDC, hFont);
-  int ret = ::GetTextFaceA(m_hDC, 100, facebuf);
+  ::GetTextFaceA(m_hDC, 100, facebuf);
   ::SelectObject(m_hDC, hOldFont);
   if (face.EqualNoCase(facebuf)) {
     return hFont;
@@ -536,7 +536,6 @@ FX_BOOL CGdiDeviceDriver::GDI_SetDIBits(const CFX_DIBitmap* pBitmap1,
       return FALSE;
     }
     int width = pSrcRect->Width(), height = pSrcRect->Height();
-    int pitch = pBitmap->GetPitch();
     LPBYTE pBuffer = pBitmap->GetBuffer();
     CFX_ByteString info = CFX_WindowsDIB::GetBitmapInfo(pBitmap);
     ((BITMAPINFOHEADER*)info.c_str())->biHeight *= -1;
@@ -556,7 +555,6 @@ FX_BOOL CGdiDeviceDriver::GDI_SetDIBits(const CFX_DIBitmap* pBitmap1,
       return FALSE;
     }
     int width = pSrcRect->Width(), height = pSrcRect->Height();
-    int pitch = pBitmap->GetPitch();
     LPBYTE pBuffer = pBitmap->GetBuffer();
     CFX_ByteString info = CFX_WindowsDIB::GetBitmapInfo(pBitmap);
     ::SetDIBitsToDevice(m_hDC, left, top, width, height, pSrcRect->left,
index 4beef9a..2022a5f 100644 (file)
@@ -484,7 +484,6 @@ static CFX_DIBitmap* _StretchMonoToGray(int dest_width,
   int src_height = pSource->GetHeight();
   int src_count = src_width * src_height;
   int dest_count = dest_width * dest_height;
-  int ratio = 255 * dest_count / src_count;
   int y_unit = src_height / dest_height;
   int x_unit = src_width / dest_width;
   int area_unit = y_unit * x_unit;
index 18fe1eb..0f75d49 100644 (file)
@@ -138,8 +138,6 @@ static CFX_DIBitmap* Transform1bppBitmap(const CFX_DIBSource* pSrc,
          pSrc->GetFormat() == FXDIB_1bppCmyk);
   CFX_FloatRect unit_rect = pDestMatrix->GetUnitRect();
   FX_RECT full_rect = unit_rect.GetOutterRect();
-  int full_left = full_rect.left;
-  int full_top = full_rect.top;
   CFX_DIBExtractor src_bitmap(pSrc);
   CFX_DIBitmap* pSrcBitmap = src_bitmap;
   if (pSrcBitmap == NULL) {
@@ -297,7 +295,7 @@ void CPSOutput::OutputPS(const FX_CHAR* string, int len) {
     int send_len = len > 1024 ? 1024 : len;
     *(FX_WORD*)m_pBuf = send_len;
     FXSYS_memcpy(m_pBuf + 2, string + sent_len, send_len);
-    int ret = ExtEscape(m_hDC, PASSTHROUGH, send_len + 2, m_pBuf, 0, NULL);
+    ExtEscape(m_hDC, PASSTHROUGH, send_len + 2, m_pBuf, 0, NULL);
     sent_len += send_len;
     len -= send_len;
   }
index 5637dbb..a7187de 100644 (file)
@@ -43,7 +43,7 @@
     'msvs_disabled_warnings': [
       4005, 4018, 4146, 4333, 4345, 4267,
       # TODO(thestig): Fix all instances, remove this, pdfium:29
-      4245, 4389, 4701, 4706, 4800,
+      4245, 4310, 4389, 4701, 4702, 4706, 4800,
     ],
     'variables': {
       'clang_warning_flags': [