commit e75704184e69dcb96be01c23d4051ae2d55744e0
author Tom Sepez <tsepez@chromium.org> Mon May 17 17:40:49 2021 +0000
committer Chromium commit bot <commit-bot@chromium.org> Mon May 17 17:40:49 2021 +0000
tree 0bb65594265c7799829dc79d7f5cd4715b6b4460
parent 6677044a60731d6ad10b8c7bb4a7808e0927cf23

Parenthesize expression in CJPX_Decoder::Decode

Otherwise, git cl format --full tries to re-write this as
   if (pitch<(m_Image->comps[0].w * 8 * m_Image->numcomps + 31)> > 5 << 2)

which doesn't compile due to the stray space.

Change-Id: Ia741b70efd5902917d177bfeea9165f6451d946e
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/80451
Auto-Submit: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>

core/fxcodec/jpx/cjpx_decoder.cpp

diff --git a/core/fxcodec/jpx/cjpx_decoder.cpp b/core/fxcodec/jpx/cjpx_decoder.cpp
index dbc1b10..8f05cb8 100644
--- a/core/fxcodec/jpx/cjpx_decoder.cpp
+++ b/core/fxcodec/jpx/cjpx_decoder.cpp
@@ -506,7 +506,7 @@
   if (m_Image->comps[0].w != m_Image->x1 || m_Image->comps[0].h != m_Image->y1)
     return false;
 
-  if (pitch<(m_Image->comps[0].w * 8 * m_Image->numcomps + 31)>> 5 << 2)
+  if (pitch < ((m_Image->comps[0].w * 8 * m_Image->numcomps + 31) >> 5) << 2)
     return false;
 
   if (swap_rgb && m_Image->numcomps < 3)