Index: xfa/fxfa/parser/cxfa_widgetdata.cpp |
diff --git a/xfa/fxfa/parser/cxfa_widgetdata.cpp b/xfa/fxfa/parser/cxfa_widgetdata.cpp |
index 7ca020fe6fd2123e304b571629c551522d2fcbf3..6ef88cdc741b55c90cb0570fdc9b1da9aecdfcf1 100644 |
--- a/xfa/fxfa/parser/cxfa_widgetdata.cpp |
+++ b/xfa/fxfa/parser/cxfa_widgetdata.cpp |
@@ -888,19 +888,19 @@ void CXFA_WidgetData::InsertItem(const CFX_WideString& wsLabel, |
if (iCount < 1) { |
CXFA_Node* pItems = m_pNode->CreateSamePacketNode(XFA_ELEMENT_Items); |
m_pNode->InsertChild(-1, pItems); |
- InsertListTextItem(pItems, wsLabel.AsStringC(), nIndex); |
+ InsertListTextItem(pItems, wsLabel, nIndex); |
CXFA_Node* pSaveItems = m_pNode->CreateSamePacketNode(XFA_ELEMENT_Items); |
m_pNode->InsertChild(-1, pSaveItems); |
pSaveItems->SetBoolean(XFA_ATTRIBUTE_Save, TRUE); |
- InsertListTextItem(pSaveItems, wsNewValue.AsStringC(), nIndex); |
+ InsertListTextItem(pSaveItems, wsNewValue, nIndex); |
} else if (iCount > 1) { |
for (int32_t i = 0; i < 2; i++) { |
CXFA_Node* pNode = listitems[i]; |
FX_BOOL bHasSave = pNode->GetBoolean(XFA_ATTRIBUTE_Save); |
if (bHasSave) |
- InsertListTextItem(pNode, wsNewValue.AsStringC(), nIndex); |
+ InsertListTextItem(pNode, wsNewValue, nIndex); |
else |
- InsertListTextItem(pNode, wsLabel.AsStringC(), nIndex); |
+ InsertListTextItem(pNode, wsLabel, nIndex); |
} |
} else { |
CXFA_Node* pNode = listitems[0]; |
@@ -916,12 +916,12 @@ void CXFA_WidgetData::InsertItem(const CFX_WideString& wsLabel, |
while (pListNode) { |
CFX_WideString wsOldValue; |
pListNode->TryContent(wsOldValue); |
- InsertListTextItem(pSaveItems, wsOldValue.AsStringC(), i); |
+ InsertListTextItem(pSaveItems, wsOldValue, i); |
i++; |
pListNode = pListNode->GetNodeItem(XFA_NODEITEM_NextSibling); |
} |
- InsertListTextItem(pNode, wsLabel.AsStringC(), nIndex); |
- InsertListTextItem(pSaveItems, wsNewValue.AsStringC(), nIndex); |
+ InsertListTextItem(pNode, wsLabel, nIndex); |
+ InsertListTextItem(pSaveItems, wsNewValue, nIndex); |
} |
if (!bNotify) |
return; |
@@ -1074,7 +1074,8 @@ int32_t CXFA_WidgetData::GetNumberOfCells() { |
CFX_WideString CXFA_WidgetData::GetBarcodeType() { |
CXFA_Node* pUIChild = GetUIChild(); |
- return pUIChild ? pUIChild->GetCData(XFA_ATTRIBUTE_Type) : NULL; |
+ return pUIChild ? CFX_WideString(pUIChild->GetCData(XFA_ATTRIBUTE_Type)) |
+ : nullptr; |
} |
FX_BOOL CXFA_WidgetData::GetBarcodeAttribute_CharEncoding(int32_t& val) { |
@@ -1084,7 +1085,8 @@ FX_BOOL CXFA_WidgetData::GetBarcodeAttribute_CharEncoding(int32_t& val) { |
if (wsCharEncoding.CompareNoCase(L"UTF-16")) { |
val = CHAR_ENCODING_UNICODE; |
return TRUE; |
- } else if (wsCharEncoding.CompareNoCase(L"UTF-8")) { |
+ } |
+ if (wsCharEncoding.CompareNoCase(L"UTF-8")) { |
val = CHAR_ENCODING_UTF8; |
return TRUE; |
} |
@@ -1475,8 +1477,8 @@ IFX_Locale* CXFA_WidgetData::GetLocal() { |
if (wsLocaleName == FX_WSTRC(L"ambient")) { |
pLocale = m_pNode->GetDocument()->GetLocalMgr()->GetDefLocale(); |
} else { |
- pLocale = m_pNode->GetDocument()->GetLocalMgr()->GetLocaleByName( |
- wsLocaleName.AsStringC()); |
+ pLocale = |
+ m_pNode->GetDocument()->GetLocalMgr()->GetLocaleByName(wsLocaleName); |
} |
} |
return pLocale; |
@@ -1739,7 +1741,7 @@ void CXFA_WidgetData::SyncValue(const CFX_WideString& wsValue, |
} |
void CXFA_WidgetData::InsertListTextItem(CXFA_Node* pItems, |
- const CFX_WideStringC& wsText, |
+ const CFX_WideString& wsText, |
int32_t nIndex) { |
CXFA_Node* pText = pItems->CreateSamePacketNode(XFA_ELEMENT_Text); |
pItems->InsertChild(nIndex, pText); |