Continue fixing FX_BOOL / int noise

core/fxcodec/jbig2/JBig2_Context.cpp

Index: core/fxcodec/jbig2/JBig2_Context.cpp
diff --git a/core/fxcodec/jbig2/JBig2_Context.cpp b/core/fxcodec/jbig2/JBig2_Context.cpp
index 42392236696a8da063b9ab47c07a261fb4268793..a45f40ed317f3a1baaadc7f72aeb71b61b30c139 100644
--- a/core/fxcodec/jbig2/JBig2_Context.cpp
+++ b/core/fxcodec/jbig2/JBig2_Context.cpp
@@ -30,8 +30,8 @@ size_t GetHuffContextSize(uint8_t val) {
   return val == 0 ? 65536 : val == 1 ? 8192 : 1024;
 }
 
-size_t GetRefAggContextSize(FX_BOOL val) {
-  return val ? 1024 : 8192;
+size_t GetRefAggContextSize(int bVal) {

[Lei Zhang, 2016/10/28 18:00:52]

Why can't this be a bool?

[Tom Sepez, 2016/10/28 18:04:44]

It's only called with things that are ints, so we'd be sticking a bunch of !!s
in several places.  Would you prefer that?

[Lei Zhang, 2016/10/28 18:09:50]

All those ints can be converted to bools: GRTEMPLATE, SBRTEMPLATE, and
SDRTEMPLATE. Want to leave this one alone and do a separate CL?

+  return bVal ? 1024 : 8192;
 }
 
 }  // namespace