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

Unified Diff: core/src/fxcrt/fx_basic_util.cpp

Issue 1520063002: Get rid of most instance of 'foo == NULL' (Closed) Base URL: https://pdfium.googlesource.com/pdfium@bstr_isnull
Patch Set: rebase Created 5 years 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 | « core/src/fxcrt/fx_basic_maps.cpp ('k') | core/src/fxcrt/fx_basic_wstring.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: core/src/fxcrt/fx_basic_util.cpp
diff --git a/core/src/fxcrt/fx_basic_util.cpp b/core/src/fxcrt/fx_basic_util.cpp
index b4c7064da2c945e806506ceef74a4c06ac831c9f..9f32ebe7520b9f305731c08db29e0d8e310933db 100644
--- a/core/src/fxcrt/fx_basic_util.cpp
+++ b/core/src/fxcrt/fx_basic_util.cpp
@@ -20,7 +20,7 @@ CFX_PrivateData::~CFX_PrivateData() {
ClearAll();
}
void FX_PRIVATEDATA::FreeData() {
- if (m_pData == NULL) {
+ if (!m_pData) {
return;
}
if (m_bSelfDestruct) {
@@ -33,7 +33,7 @@ void CFX_PrivateData::AddData(void* pModuleId,
void* pData,
PD_CALLBACK_FREEDATA callback,
FX_BOOL bSelfDestruct) {
- if (pModuleId == NULL) {
+ if (!pModuleId) {
return;
}
FX_PRIVATEDATA* pList = m_DataList.GetData();
@@ -58,7 +58,7 @@ void CFX_PrivateData::SetPrivateObj(void* pModuleId, CFX_DestructObject* pObj) {
AddData(pModuleId, pObj, NULL, TRUE);
}
FX_BOOL CFX_PrivateData::RemovePrivateData(void* pModuleId) {
- if (pModuleId == NULL) {
+ if (!pModuleId) {
return FALSE;
}
FX_PRIVATEDATA* pList = m_DataList.GetData();
@@ -72,7 +72,7 @@ FX_BOOL CFX_PrivateData::RemovePrivateData(void* pModuleId) {
return FALSE;
}
void* CFX_PrivateData::GetPrivateData(void* pModuleId) {
- if (pModuleId == NULL) {
+ if (!pModuleId) {
return NULL;
}
FX_PRIVATEDATA* pList = m_DataList.GetData();
@@ -93,7 +93,7 @@ void CFX_PrivateData::ClearAll() {
m_DataList.RemoveAll();
}
void FX_atonum(const CFX_ByteStringC& strc, FX_BOOL& bInteger, void* pData) {
- if (FXSYS_memchr(strc.GetPtr(), '.', strc.GetLength()) == NULL) {
+ if (!FXSYS_memchr(strc.GetPtr(), '.', strc.GetLength())) {
bInteger = TRUE;
int cc = 0, integer = 0;
const FX_CHAR* str = strc.GetCStr();
@@ -257,7 +257,7 @@ void* FX_OpenFolder(const FX_WCHAR* path) {
FX_BOOL FX_GetNextFile(void* handle,
CFX_ByteString& filename,
FX_BOOL& bFolder) {
- if (handle == NULL) {
+ if (!handle) {
return FALSE;
}
#if _FXM_PLATFORM_ == _FXM_PLATFORM_WINDOWS_
@@ -289,7 +289,7 @@ FX_BOOL FX_GetNextFile(void* handle,
return FALSE;
#else
struct dirent* de = readdir((DIR*)handle);
- if (de == NULL) {
+ if (!de) {
return FALSE;
}
filename = de->d_name;
@@ -300,7 +300,7 @@ FX_BOOL FX_GetNextFile(void* handle,
FX_BOOL FX_GetNextFile(void* handle,
CFX_WideString& filename,
FX_BOOL& bFolder) {
- if (handle == NULL) {
+ if (!handle) {
return FALSE;
}
#if _FXM_PLATFORM_ == _FXM_PLATFORM_WINDOWS_
@@ -319,7 +319,7 @@ FX_BOOL FX_GetNextFile(void* handle,
return FALSE;
#else
struct dirent* de = readdir((DIR*)handle);
- if (de == NULL) {
+ if (!de) {
return FALSE;
}
filename = CFX_WideString::FromLocal(de->d_name);
@@ -328,7 +328,7 @@ FX_BOOL FX_GetNextFile(void* handle,
#endif
}
void FX_CloseFolder(void* handle) {
- if (handle == NULL) {
+ if (!handle) {
return;
}
#if _FXM_PLATFORM_ == _FXM_PLATFORM_WINDOWS_
« no previous file with comments | « core/src/fxcrt/fx_basic_maps.cpp ('k') | core/src/fxcrt/fx_basic_wstring.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698