XFA: Remove cond ? TRUE : FALSE.
authorLei Zhang <thestig@chromium.org>
Tue, 20 Oct 2015 17:21:36 +0000 (10:21 -0700)
committerLei Zhang <thestig@chromium.org>
Tue, 20 Oct 2015 17:21:36 +0000 (10:21 -0700)
R=tsepez@chromium.org

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

13 files changed:
core/src/fxcodec/codec/fx_codec_progress.cpp
xfa/src/fdp/src/fde/fde_gedevice.cpp
xfa/src/fdp/src/tto/fde_textout.cpp
xfa/src/fwl/src/basewidget/fwl_comboboximp.cpp
xfa/src/fwl/src/basewidget/fwl_datetimepickerimp.cpp
xfa/src/fwl/src/core/fwl_formimp.cpp
xfa/src/fxfa/src/parser/xfa_document_datamerger_imp.cpp
xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp
xfa/src/fxfa/src/parser/xfa_object_imp.cpp
xfa/src/fxfa/src/parser/xfa_objectacc_imp.cpp
xfa/src/fxfa/src/parser/xfa_script_imp.cpp
xfa/src/fxfa/src/parser/xfa_script_resolveprocessor.cpp
xfa/src/fxjse/src/class.cpp

index 0ad4c2a..885b751 100644 (file)
@@ -716,7 +716,7 @@ void CCodec_ProgressiveDecoder::GifReadScanlineCallback(void* pModule,
     pal_index = pCodec->m_GifTransIndex;\r
   }\r
   FXSYS_memset(pCodec->m_pDecodeBuf, pal_index, pCodec->m_SrcWidth);\r
-  FX_BOOL bLastPass = ((row_num % 2) == 1) ? TRUE : FALSE;\r
+  FX_BOOL bLastPass = (row_num % 2) == 1;\r
   int32_t line = row_num + pCodec->m_GifFrameRect.top;\r
   int32_t left = pCodec->m_GifFrameRect.left;\r
   FXSYS_memcpy(pCodec->m_pDecodeBuf + left, row_buf, img_width);\r
index 4b716b4..c50fcca 100644 (file)
@@ -169,8 +169,7 @@ FX_BOOL CFDE_FxgeDevice::DrawString(IFDE_Brush* pBrush,
       SubstFxFont.m_Weight = dwFontStyle & FX_FONTSTYLE_Bold ? 700 : 400;\r
       SubstFxFont.m_WeightCJK = SubstFxFont.m_Weight;\r
       SubstFxFont.m_ItalicAngle = dwFontStyle & FX_FONTSTYLE_Italic ? -12 : 0;\r
-      SubstFxFont.m_bItlicCJK =\r
-          dwFontStyle & FX_FONTSTYLE_Italic ? TRUE : FALSE;\r
+      SubstFxFont.m_bItlicCJK = !!(dwFontStyle & FX_FONTSTYLE_Italic);\r
 #endif\r
       for (int32_t i = 0; i < iCount; ++i) {\r
         pSTFont = pFont->GetSubstFont((int32_t)pCP->m_GlyphIndex);\r
index 4661bd6..d7e1762 100644 (file)
@@ -232,8 +232,8 @@ void CFDE_TextOut::CalcTextSize(const FX_WCHAR* pwsStr,
   SetLineWidth(rect);\r
   m_iTotalLines = 0;\r
   const FX_WCHAR* pStr = pwsStr;\r
-  FX_BOOL bHotKey = (m_dwStyles & FDE_TTOSTYLE_HotKey) ? TRUE : FALSE;\r
-  FX_BOOL bVertical = (m_dwStyles & FDE_TTOSTYLE_VerticalLayout) ? TRUE : FALSE;\r
+  FX_BOOL bHotKey = !!(m_dwStyles & FDE_TTOSTYLE_HotKey);\r
+  FX_BOOL bVertical = !!(m_dwStyles & FDE_TTOSTYLE_VerticalLayout);\r
   FX_FLOAT fWidth = 0.0f;\r
   FX_FLOAT fHeight = 0.0f;\r
   FX_FLOAT fStartPos = bVertical ? rect.bottom() : rect.right();\r
@@ -313,7 +313,7 @@ FX_BOOL CFDE_TextOut::RetrieveLineWidth(FX_DWORD dwBreakStatus,
   }\r
   FX_FLOAT fLineStep =\r
       (m_fLineSpace > m_fFontSize) ? m_fLineSpace : m_fFontSize;\r
-  FX_BOOL bLineWrap = (m_dwStyles & FDE_TTOSTYLE_LineWrap) ? TRUE : FALSE;\r
+  FX_BOOL bLineWrap = !!(m_dwStyles & FDE_TTOSTYLE_LineWrap);\r
   FX_FLOAT fLineWidth = 0.0f;\r
   int32_t iCount = m_pTxtBreak->CountBreakPieces();\r
   for (int32_t i = 0; i < iCount; i++) {\r
@@ -501,9 +501,9 @@ void CFDE_TextOut::LoadText(const FX_WCHAR* pwsStr,
   FX_WCHAR* pStr = m_wsText.GetBuffer(iLength);\r
   int32_t iTxtLength = iLength;\r
   ExpandBuffer(iTxtLength, 0);\r
-  FX_BOOL bHotKey = (m_dwStyles & FDE_TTOSTYLE_HotKey) ? TRUE : FALSE;\r
-  FX_BOOL bVertical = (m_dwStyles & FDE_TTOSTYLE_VerticalLayout) ? TRUE : FALSE;\r
-  FX_BOOL bLineWrap = (m_dwStyles & FDE_TTOSTYLE_LineWrap) ? TRUE : FALSE;\r
+  FX_BOOL bHotKey = !!(m_dwStyles & FDE_TTOSTYLE_HotKey);\r
+  FX_BOOL bVertical = !!(m_dwStyles & FDE_TTOSTYLE_VerticalLayout);\r
+  FX_BOOL bLineWrap = !!(m_dwStyles & FDE_TTOSTYLE_LineWrap);\r
   FX_FLOAT fLineStep =\r
       (m_fLineSpace > m_fFontSize) ? m_fLineSpace : m_fFontSize;\r
   FX_FLOAT fLineStop = bVertical ? rect.left : rect.bottom();\r
@@ -564,9 +564,9 @@ FX_BOOL CFDE_TextOut::RetriecePieces(FX_DWORD dwBreakStatus,
                                      int32_t& iPieceWidths,\r
                                      FX_BOOL bReload,\r
                                      const CFX_RectF& rect) {\r
-  FX_BOOL bSingleLine = (m_dwStyles & FDE_TTOSTYLE_SingleLine) ? TRUE : FALSE;\r
-  FX_BOOL bLineWrap = (m_dwStyles & FDE_TTOSTYLE_LineWrap) ? TRUE : FALSE;\r
-  FX_BOOL bVertical = (m_dwStyles & FDE_TTOSTYLE_VerticalLayout) ? TRUE : FALSE;\r
+  FX_BOOL bSingleLine = !!(m_dwStyles & FDE_TTOSTYLE_SingleLine);\r
+  FX_BOOL bLineWrap = !!(m_dwStyles & FDE_TTOSTYLE_LineWrap);\r
+  FX_BOOL bVertical = !!(m_dwStyles & FDE_TTOSTYLE_VerticalLayout);\r
   FX_FLOAT fLineStep =\r
       (m_fLineSpace > m_fFontSize) ? m_fLineSpace : m_fFontSize;\r
   if (bVertical) {\r
@@ -709,7 +709,7 @@ void CFDE_TextOut::Reload(const CFX_RectF& rect) {
 }\r
 void CFDE_TextOut::ReloadLinePiece(CFDE_TTOLine* pLine, const CFX_RectF& rect) {\r
   const FX_WCHAR* pwsStr = (const FX_WCHAR*)m_wsText;\r
-  FX_BOOL bVertical = (m_dwStyles & FDE_TTOSTYLE_VerticalLayout) ? TRUE : FALSE;\r
+  FX_BOOL bVertical = !!(m_dwStyles & FDE_TTOSTYLE_VerticalLayout);\r
   int32_t iPieceWidths = 0;\r
   FDE_LPTTOPIECE pPiece = pLine->GetPtrAt(0);\r
   int32_t iStartChar = pPiece->iStartChar;\r
@@ -739,7 +739,7 @@ void CFDE_TextOut::ReloadLinePiece(CFDE_TTOLine* pLine, const CFX_RectF& rect) {
   m_pTxtBreak->Reset();\r
 }\r
 void CFDE_TextOut::DoAlignment(const CFX_RectF& rect) {\r
-  FX_BOOL bVertical = (m_dwStyles & FDE_TTOSTYLE_VerticalLayout) ? TRUE : FALSE;\r
+  FX_BOOL bVertical = !!(m_dwStyles & FDE_TTOSTYLE_VerticalLayout);\r
   FX_FLOAT fLineStopS = bVertical ? rect.right() : rect.bottom();\r
   int32_t iLines = m_ttoLines.GetSize();\r
   if (iLines < 1) {\r
@@ -842,10 +842,10 @@ void CFDE_TextOut::ToTextRun(const FDE_LPTTOPIECE pPiece, FX_TXTRUN& tr) {
   tr.pRect = &pPiece->rtPiece;\r
 }\r
 void CFDE_TextOut::DrawLine(const FDE_LPTTOPIECE pPiece, IFDE_Pen*& pPen) {\r
-  FX_BOOL bUnderLine = (m_dwStyles & FDE_TTOSTYLE_Underline) ? TRUE : FALSE;\r
-  FX_BOOL bStrikeOut = (m_dwStyles & FDE_TTOSTYLE_Strikeout) ? TRUE : FALSE;\r
-  FX_BOOL bHotKey = (m_dwStyles & FDE_TTOSTYLE_HotKey) ? TRUE : FALSE;\r
-  FX_BOOL bVertical = (m_dwStyles & FDE_TTOSTYLE_VerticalLayout) ? TRUE : FALSE;\r
+  FX_BOOL bUnderLine = !!(m_dwStyles & FDE_TTOSTYLE_Underline);\r
+  FX_BOOL bStrikeOut = !!(m_dwStyles & FDE_TTOSTYLE_Strikeout);\r
+  FX_BOOL bHotKey = !!(m_dwStyles & FDE_TTOSTYLE_HotKey);\r
+  FX_BOOL bVertical = !!(m_dwStyles & FDE_TTOSTYLE_VerticalLayout);\r
   if (!bUnderLine && !bStrikeOut && !bHotKey) {\r
     return;\r
   }\r
index 4460a8d..64e9b40 100644 (file)
@@ -869,13 +869,13 @@ FX_BOOL CFWL_ComboBoxImp::EditPaste(const CFX_WideString& wsPaste) {
   return m_pEdit->Paste(wsPaste);\r
 }\r
 FX_BOOL CFWL_ComboBoxImp::EditSelectAll() {\r
-  return (m_pEdit->AddSelRange(0) == FWL_ERR_Succeeded) ? TRUE : FALSE;\r
+  return m_pEdit->AddSelRange(0) == FWL_ERR_Succeeded;\r
 }\r
 FX_BOOL CFWL_ComboBoxImp::EditDelete() {\r
-  return (m_pEdit->ClearText() == FWL_ERR_Succeeded) ? TRUE : FALSE;\r
+  return m_pEdit->ClearText() == FWL_ERR_Succeeded;\r
 }\r
 FX_BOOL CFWL_ComboBoxImp::EditDeSelect() {\r
-  return (m_pEdit->ClearSelections() == FWL_ERR_Succeeded) ? TRUE : FALSE;\r
+  return m_pEdit->ClearSelections() == FWL_ERR_Succeeded;\r
 }\r
 FWL_ERR CFWL_ComboBoxImp::GetBBox(CFX_RectF& rect) {\r
   if (m_pWidgetMgr->IsFormDisabled()) {\r
index c378ccd..866dc49 100644 (file)
@@ -662,13 +662,13 @@ FX_BOOL CFWL_DateTimePickerImp::Paste(const CFX_WideString& wsPaste) {
   return m_pEdit->Paste(wsPaste);\r
 }\r
 FX_BOOL CFWL_DateTimePickerImp::SelectAll() {\r
-  return (m_pEdit->AddSelRange(0) == FWL_ERR_Succeeded) ? TRUE : FALSE;\r
+  return m_pEdit->AddSelRange(0) == FWL_ERR_Succeeded;\r
 }\r
 FX_BOOL CFWL_DateTimePickerImp::Delete() {\r
-  return (m_pEdit->ClearText() == FWL_ERR_Succeeded) ? TRUE : FALSE;\r
+  return m_pEdit->ClearText() == FWL_ERR_Succeeded;\r
 }\r
 FX_BOOL CFWL_DateTimePickerImp::DeSelect() {\r
-  return (m_pEdit->ClearSelections() == FWL_ERR_Succeeded) ? TRUE : FALSE;\r
+  return m_pEdit->ClearSelections() == FWL_ERR_Succeeded;\r
 }\r
 FWL_ERR CFWL_DateTimePickerImp::GetBBox(CFX_RectF& rect) {\r
   if (m_pWidgetMgr->IsFormDisabled()) {\r
index 85139ca..2facbf3 100644 (file)
@@ -885,7 +885,7 @@ void CFWL_FormImp::SetThemeData() {
 }\r
 FX_BOOL CFWL_FormImp::HasIcon() {\r
   IFWL_FormDP* pData = (IFWL_FormDP*)m_pProperties->m_pDataProvider;\r
-  return pData->GetIcon(m_pInterface, FALSE) ? TRUE : FALSE;\r
+  return !!pData->GetIcon(m_pInterface, FALSE);\r
 }\r
 void CFWL_FormImp::UpdateIcon() {\r
   CFWL_WidgetMgr* pWidgetMgr = (CFWL_WidgetMgr*)FWL_GetWidgetMgr();\r
index 42652ad..8da9184 100644 (file)
@@ -1201,7 +1201,7 @@ static void XFA_DataMerge_UpdateBindingRelations(CXFA_Document* pDocument,
           if (pDataNode) {\r
             XFA_DataMerge_CreateDataBinding(\r
                 pFormNode, pDataNode,\r
-                rs.dwFlags == XFA_RESOVENODE_RSTYPE_ExistNodes ? TRUE : FALSE);\r
+                rs.dwFlags == XFA_RESOVENODE_RSTYPE_ExistNodes);\r
           } else {\r
             XFA_DataMerge_FormValueNode_MatchNoneCreateChild(pFormNode);\r
           }\r
index 1894480..4ab1dc9 100644 (file)
@@ -1432,7 +1432,7 @@ FX_BOOL CXFA_LayoutPageMgr::GetNextAvailContentHeight(FX_FLOAT fChildHeight) {
   if (pCurContentNode) {\r
     FX_FLOAT fNextContentHeight =\r
         pCurContentNode->GetMeasure(XFA_ATTRIBUTE_H).ToUnit(XFA_UNIT_Pt);\r
-    return fNextContentHeight > fChildHeight ? TRUE : FALSE;\r
+    return fNextContentHeight > fChildHeight;\r
   }\r
   CXFA_Node* pPageNode = GetCurrentContainerRecord()->pCurPageArea->m_pFormNode;\r
   CXFA_Node* pOccurNode = pPageNode->GetFirstChildByClass(XFA_ELEMENT_Occur);\r
index db05967..0bacfcd 100644 (file)
@@ -1752,8 +1752,7 @@ void CXFA_Node::Script_Boolean_Value(FXJSE_HVALUE hValue,
     SetScriptContent(wsNewValue, wsFormatValue, TRUE, TRUE);\r
   } else {\r
     CFX_WideString wsValue = GetScriptContent(TRUE);\r
-    FXJSE_Value_SetBoolean(hValue,\r
-                           wsValue.Equal(FX_WSTRC(L"1")) ? TRUE : FALSE);\r
+    FXJSE_Value_SetBoolean(hValue, wsValue.Equal(FX_WSTRC(L"1")));\r
   }\r
 }\r
 struct XFA_ExecEventParaInfo {\r
@@ -5449,8 +5448,7 @@ CXFA_AttachNodeList::CXFA_AttachNodeList(CXFA_Document* pDocument,
 }\r
 int32_t CXFA_AttachNodeList::GetLength() {\r
   return m_pAttachNode->CountChildren(\r
-      XFA_ELEMENT_UNKNOWN,\r
-      m_pAttachNode->GetClassID() == XFA_ELEMENT_Subform ? TRUE : FALSE);\r
+      XFA_ELEMENT_UNKNOWN, m_pAttachNode->GetClassID() == XFA_ELEMENT_Subform);\r
 }\r
 FX_BOOL CXFA_AttachNodeList::Append(CXFA_Node* pNode) {\r
   CXFA_Node* pParent = pNode->GetNodeItem(XFA_NODEITEM_Parent);\r
@@ -5473,5 +5471,5 @@ FX_BOOL CXFA_AttachNodeList::Remove(CXFA_Node* pNode) {
 CXFA_Node* CXFA_AttachNodeList::Item(int32_t iIndex) {\r
   return m_pAttachNode->GetChild(\r
       iIndex, XFA_ELEMENT_UNKNOWN,\r
-      m_pAttachNode->GetClassID() == XFA_ELEMENT_Subform ? TRUE : FALSE);\r
+      m_pAttachNode->GetClassID() == XFA_ELEMENT_Subform);\r
 }\r
index ab9e333..c239ac1 100644 (file)
@@ -837,7 +837,7 @@ int32_t CXFA_Line::GetHand() {
 }\r
 FX_BOOL CXFA_Line::GetSlop() {\r
   XFA_ATTRIBUTEENUM eSlop = m_pNode->GetEnum(XFA_ATTRIBUTE_Slope);\r
-  return (eSlop == XFA_ATTRIBUTEENUM_Slash) ? TRUE : FALSE;\r
+  return eSlop == XFA_ATTRIBUTEENUM_Slash;\r
 }\r
 CXFA_Edge CXFA_Line::GetEdge() {\r
   return CXFA_Edge(m_pNode->GetChild(0, XFA_ELEMENT_Edge));\r
index f8b8f76..34668ea 100644 (file)
@@ -672,7 +672,7 @@ int32_t CXFA_ScriptContext::ResolveObjects(CXFA_Object* refNode,
       FX_BOOL bCreate =\r
           m_pResolveProcessor->GetNodeHelper()->XFA_ResolveNodes_CreateNode(\r
               rndFind.m_wsName, rndFind.m_wsCondition,\r
-              nStart == wsExpression.GetLength() ? TRUE : FALSE, this);\r
+              nStart == wsExpression.GetLength(), this);\r
       if (bCreate) {\r
         continue;\r
       } else {\r
@@ -732,7 +732,7 @@ int32_t CXFA_ScriptContext::ResolveObjects(CXFA_Object* refNode,
         FX_BOOL bCreate =\r
             m_pResolveProcessor->GetNodeHelper()->XFA_ResolveNodes_CreateNode(\r
                 rndFind.m_wsName, rndFind.m_wsCondition,\r
-                nStart == wsExpression.GetLength() ? TRUE : FALSE, this);\r
+                nStart == wsExpression.GetLength(), this);\r
         if (bCreate) {\r
           continue;\r
         } else {\r
index dfbf22d..7919395 100644 (file)
@@ -314,7 +314,7 @@ int32_t CXFA_ResolveProcessor::XFA_ResolveNodes_Normal(
         if (m_pNodeHelper->XFA_NodeIsTransparent((CXFA_Node*)curNode)) {\r
           m_pNodeHelper->XFA_CountSiblings(\r
               (CXFA_Node*)nodes[0], XFA_LOGIC_Transparent, &upArrayNodes,\r
-              dwStyles & XFA_RESOLVENODE_TagName ? TRUE : FALSE);\r
+              !!(dwStyles & XFA_RESOLVENODE_TagName));\r
         }\r
         if (upArrayNodes.GetSize() > nodes.GetSize()) {\r
           upArrayNodes[0] = (CXFA_Node*)nodes[0];\r
@@ -463,7 +463,7 @@ int32_t CXFA_ResolveProcessor::XFA_ResolveNodes_Normal(
         CXFA_NodeArray upArrayNodes;\r
         m_pNodeHelper->XFA_CountSiblings(\r
             (CXFA_Node*)nodes[0], XFA_LOGIC_Transparent, &upArrayNodes,\r
-            dwStyles & XFA_RESOLVENODE_TagName ? TRUE : FALSE);\r
+            !!(dwStyles & XFA_RESOLVENODE_TagName));\r
         if (upArrayNodes.GetSize() > nodes.GetSize()) {\r
           upArrayNodes[0] = (CXFA_Node*)nodes[0];\r
           nodes.RemoveAll();\r
index 891c7e1..04b087f 100644 (file)
@@ -183,7 +183,7 @@ FXJSE_HVALUE CFXJSE_Arguments::GetValue(int32_t index) const {
 FX_BOOL CFXJSE_Arguments::GetBoolean(int32_t index) const {\r
   const CFXJSE_ArgumentsImpl* lpArguments =\r
       reinterpret_cast<const CFXJSE_ArgumentsImpl* const>(this);\r
-  return (*lpArguments->m_pInfo)[index]->BooleanValue() ? TRUE : FALSE;\r
+  return (*lpArguments->m_pInfo)[index]->BooleanValue();\r
 }\r
 int32_t CFXJSE_Arguments::GetInt32(int32_t index) const {\r
   const CFXJSE_ArgumentsImpl* lpArguments =\r