Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(300)

Unified Diff: xfa/fxfa/parser/xfa_script_hostpseudomodel.cpp

Issue 2031873003: Get rid of NULLs in xfa/ (Closed) Base URL: https://pdfium.googlesource.com/pdfium@nullptr_fpdfsdk
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « xfa/fxfa/parser/xfa_script.h ('k') | xfa/fxfa/parser/xfa_script_imp.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/fxfa/parser/xfa_script_hostpseudomodel.cpp
diff --git a/xfa/fxfa/parser/xfa_script_hostpseudomodel.cpp b/xfa/fxfa/parser/xfa_script_hostpseudomodel.cpp
index 3adabe6a6ed63b48d72b00e71cdebaf223de08ee..9ade41ede6b32447d95f2395ae726fc2ce7a51af 100644
--- a/xfa/fxfa/parser/xfa_script_hostpseudomodel.cpp
+++ b/xfa/fxfa/parser/xfa_script_hostpseudomodel.cpp
@@ -313,7 +313,7 @@ void CScript_HostPseudoModel::Script_HostPseudoModel_OpenList(
if (!pNotify) {
return;
}
- CXFA_Node* pNode = NULL;
+ CXFA_Node* pNode = nullptr;
if (iLength >= 1) {
std::unique_ptr<CFXJSE_Value> pValue(pArguments->GetValue(0));
if (FXJSE_Value_IsObject(pValue.get())) {
@@ -449,7 +449,7 @@ void CScript_HostPseudoModel::Script_HostPseudoModel_ResetData(
}
int32_t iStart = 0;
CFX_WideString wsName;
- CXFA_Node* pNode = NULL;
+ CXFA_Node* pNode = nullptr;
int32_t iExpLength = wsExpression.GetLength();
while (iStart < iExpLength) {
iStart = XFA_FilterName(wsExpression.AsStringC(), iStart, wsName);
@@ -510,7 +510,7 @@ void CScript_HostPseudoModel::Script_HostPseudoModel_SetFocus(
if (!pNotify) {
return;
}
- CXFA_Node* pNode = NULL;
+ CXFA_Node* pNode = nullptr;
if (iLength >= 1) {
std::unique_ptr<CFXJSE_Value> pValue(pArguments->GetValue(0));
if (FXJSE_Value_IsObject(pValue.get())) {
@@ -605,9 +605,9 @@ FX_BOOL CScript_HostPseudoModel::Script_HostPseudoModel_ValidateArgsForMsg(
CFXJSE_Arguments* pArguments,
int32_t iArgIndex,
CFX_WideString& wsValue) {
- if (pArguments == NULL || iArgIndex < 0) {
+ if (!pArguments || iArgIndex < 0)
return FALSE;
- }
+
FX_BOOL bIsJsType = FALSE;
if (m_pDocument->GetScriptContext()->GetType() ==
XFA_SCRIPTLANGTYPE_Javascript) {
« no previous file with comments | « xfa/fxfa/parser/xfa_script.h ('k') | xfa/fxfa/parser/xfa_script_imp.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698