diff options
author | caryclark <caryclark@google.com> | 2016-05-09 10:08:18 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-05-09 10:08:18 -0700 |
commit | 690a99c20a0f4cb67a386e64d711023dd8e15bed (patch) | |
tree | 7f0d491c9b8f6dc093c420d34dc63c9006e33170 | |
parent | e00f75c2318d55dfa261f1915decd2545c44ccfb (diff) | |
download | pdfium-690a99c20a0f4cb67a386e64d711023dd8e15bed.tar.xz |
fix linux warnings
The Skia variant generates a couple of warnings on the linux build.
One of the warning silenced is an unused variable. Once PDFs are
identified that exercise this variable, the dummy code will be
replaced.
R=tsepez@chromium.org
Review-Url: https://codereview.chromium.org/1962683002
-rw-r--r-- | core/fxge/skia/fx_skia_device.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/core/fxge/skia/fx_skia_device.cpp b/core/fxge/skia/fx_skia_device.cpp index b00bc8e5c5..89a7902d51 100644 --- a/core/fxge/skia/fx_skia_device.cpp +++ b/core/fxge/skia/fx_skia_device.cpp @@ -568,6 +568,8 @@ CFX_SkiaDeviceDriver::CFX_SkiaDeviceDriver(CFX_DIBitmap* pBitmap, nullptr, /* to do : set color table */ nullptr, nullptr); m_pCanvas = new SkCanvas(skBitmap); + if (m_bGroupKnockout) + SkDebugf(""); // FIXME(caryclark) suppress 'm_bGroupKnockout is unused' } CFX_SkiaDeviceDriver::CFX_SkiaDeviceDriver(int size_x, int size_y) @@ -909,7 +911,8 @@ FX_BOOL CFX_SkiaDeviceDriver::DrawShading(const CPDF_ShadingPattern* pPattern, skClip.transform(skMatrix); } SkMatrix inverse; - skMatrix.invert(&inverse); + if (!skMatrix.invert(&inverse)) + return false; skPath.addRect(skRect); skPath.transform(inverse); } |