XFA: Fix a bunch of pointless returns.

xfa/src/fxfa/src/parser/xfa_layout_itemlayout.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include <algorithm>
 
 #include "xfa/src/foxitlib.h"
 #include "xfa/src/fxfa/src/common/xfa_utils.h"
@@ skipping 342 matching lines @@
           pChildItem, pSecondLayoutItem,
           fSplitPos - fCurTopMargin - fCurBottomMargin - pChildItem->m_sPos.y);
       fAddMarginHeight = pSecondLayoutItem->m_sSize.y - fOldHeight;
       pLayoutItem->AddChild(pChildItem);
     }
   }
 }
 void CXFA_ItemLayoutProcessor::SplitLayoutItem(FX_FLOAT fSplitPos) {
   ASSERT(m_pLayoutItem);
   SplitLayoutItem(m_pLayoutItem, NULL, fSplitPos);
-  return;
 }
 
 IXFA_LayoutPage* CXFA_LayoutItem::GetPage() const {
   for (CXFA_LayoutItem* pCurNode = const_cast<CXFA_LayoutItem*>(this); pCurNode;
        pCurNode = pCurNode->m_pParent) {
     if (pCurNode->m_pFormNode->GetClassID() == XFA_ELEMENT_PageArea)
       return static_cast<CXFA_ContainerLayoutItem*>(pCurNode);
   }
   return nullptr;
 }
@@ skipping 2644 matching lines @@
         (int32_t)(uintptr_t)m_PendingNodesCount.GetValueAt(pTemplate);
     if (iCount >= iMax) {
       return FALSE;
     }
     iCount++;
     m_PendingNodesCount.SetAt(pTemplate, (void*)(uintptr_t)(iCount));
     return TRUE;
   }
   return TRUE;
 }