Cleanup CJBig2_ArithDecoder.

Cleanup CJBig2_ArithDecoder.

R=tsepez@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/0fef7dd405ac85660dbeddd2e47620daffa70cb5

[Lei Zhang, 2015-09-28 17:29:44]

thestig@chromium.org changed reviewers:
+ tsepez@chromium.org

[Tom Sepez, 2015-09-29 17:56:50]

lgtm

https://codereview.chromium.org/1368153003/diff/20001/core/src/fxcodec/jbig2/...
File core/src/fxcodec/jbig2/JBig2_ArithDecoder.cpp (right):

https://codereview.chromium.org/1368153003/diff/20001/core/src/fxcodec/jbig2/...
core/src/fxcodec/jbig2/JBig2_ArithDecoder.cpp:79: int D = 1 - pCX->MPS;
Nit: not today, but I think D and pCX->MPS are really bools and your want:

 D= !pCX->MPS.

https://codereview.chromium.org/1368153003/diff/20001/core/src/fxcodec/jbig2/...
core/src/fxcodec/jbig2/JBig2_ArithDecoder.cpp:81: pCX->MPS = 1 - pCX->MPS;
nit: same here.

[Lei Zhang, 2015-09-30 04:12:05]

Patchset #3 (id:40001) has been deleted

[Lei Zhang, 2015-09-30 04:14:44]

https://codereview.chromium.org/1368153003/diff/20001/core/src/fxcodec/jbig2/...
File core/src/fxcodec/jbig2/JBig2_ArithDecoder.cpp (right):

https://codereview.chromium.org/1368153003/diff/20001/core/src/fxcodec/jbig2/...
core/src/fxcodec/jbig2/JBig2_ArithDecoder.cpp:79: int D = 1 - pCX->MPS;
On 2015/09/29 17:56:50, Tom Sepez wrote:
> Nit: not today, but I think D and pCX->MPS are really bools and your want:
> 
>  D= !pCX->MPS.

Added a TODO.

[Lei Zhang, 2015-09-30 04:15:03]

Committed patchset #3 (id:60001) manually as
0fef7dd405ac85660dbeddd2e47620daffa70cb5 (presubmit successful).