Add missing parenthesis in FXARGB_GETDIB macro
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
diff --git a/AUTHORS b/AUTHORS
index 4dc71c8..332b853 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -26,6 +26,7 @@
Reid Kleckner <rnk@chromium.org>
Robert Sesek <rsesek@chromium.org>
Thomas Sepez <tsepez@chromium.org>
+Jiang Jiang <jiangj@opera.com>
Foxit Software Inc <*@foxitsoftware.com>
Google Inc. <*@google.com>
diff --git a/core/include/fxge/fx_dib.h b/core/include/fxge/fx_dib.h
index 957a957..d8a2003 100644
--- a/core/include/fxge/fx_dib.h
+++ b/core/include/fxge/fx_dib.h
@@ -105,7 +105,7 @@
((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), \