Add missing parenthesis in FXARGB_GETDIB macro
authorBo Xu <bo_xu@foxitsoftware.com>
Tue, 21 Oct 2014 19:04:41 +0000 (12:04 -0700)
committerBo Xu <bo_xu@foxitsoftware.com>
Tue, 21 Oct 2014 19:04:41 +0000 (12:04 -0700)
You can get warning when using FXARGB_GETDIB() with & operation in
the same statement like: FXARGB_GETDIB(src_scan) & 0xffffff in
fx_dib_composite.cpp:

../../third_party/pdfium/core/src/fxge/dib/fx_dib_composite.cpp:737:205:
error: '&' within '|' [-Werror,-Wbitwise-op-parentheses]

Original patch from jiangj@opera.com

R=thakis@chromium.org

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

AUTHORS
core/include/fxge/fx_dib.h

diff --git a/AUTHORS b/AUTHORS
index 4dc71c8..332b853 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -26,6 +26,7 @@ Raymes Khoury <raymes@chromium.org>
 Reid Kleckner <rnk@chromium.org>\r
 Robert Sesek <rsesek@chromium.org>\r
 Thomas Sepez <tsepez@chromium.org>\r
+Jiang Jiang <jiangj@opera.com>\r
 \r
 Foxit Software Inc <*@foxitsoftware.com>\r
 Google Inc. <*@google.com>\r
index 957a957..d8a2003 100644 (file)
@@ -105,7 +105,7 @@ FX_ARGB ArgbEncode(int a, FX_COLORREF rgb);
         ((FX_LPBYTE)(p))[1] = (FX_BYTE)((cmyk) >> 16), \
                               ((FX_LPBYTE)(p))[2] = (FX_BYTE)((cmyk) >> 8), \
                                       ((FX_LPBYTE)(p))[3] = (FX_BYTE)(cmyk))
-#define FXARGB_GETDIB(p) (((FX_LPBYTE)(p))[0]) | (((FX_LPBYTE)(p))[1] << 8) | (((FX_LPBYTE)(p))[2] << 16) | (((FX_LPBYTE)(p))[3] << 24)
+#define FXARGB_GETDIB(p) ((((FX_LPBYTE)(p))[0]) | (((FX_LPBYTE)(p))[1] << 8) | (((FX_LPBYTE)(p))[2] << 16) | (((FX_LPBYTE)(p))[3] << 24))
 #define FXARGB_SETDIB(p, argb) ((FX_LPBYTE)(p))[0] = (FX_BYTE)(argb), \
         ((FX_LPBYTE)(p))[1] = (FX_BYTE)((argb) >> 8), \
                               ((FX_LPBYTE)(p))[2] = (FX_BYTE)((argb) >> 16), \