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

Unified Diff: src/core/SkResourceCache.cpp

Issue 1316233002: Style Change: NULL->nullptr (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: 2015-08-27 (Thursday) 10:25:06 EDT Created 5 years, 4 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 | « src/core/SkResourceCache.h ('k') | src/core/SkScalar.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkResourceCache.cpp
diff --git a/src/core/SkResourceCache.cpp b/src/core/SkResourceCache.cpp
index 911a09071eee582a979d7c8ef722aac34e204552..4d2453c71e1bb3dc7a67dee82a59a2ac7a83c6c0 100644
--- a/src/core/SkResourceCache.cpp
+++ b/src/core/SkResourceCache.cpp
@@ -59,17 +59,17 @@ class SkResourceCache::Hash :
///////////////////////////////////////////////////////////////////////////////
void SkResourceCache::init() {
- fHead = NULL;
- fTail = NULL;
+ fHead = nullptr;
+ fTail = nullptr;
fHash = new Hash;
fTotalBytesUsed = 0;
fCount = 0;
fSingleAllocationByteLimit = 0;
- fAllocator = NULL;
+ fAllocator = nullptr;
// One of these should be explicit set by the caller after we return.
fTotalByteLimit = 0;
- fDiscardableFactory = NULL;
+ fDiscardableFactory = nullptr;
}
#include "SkDiscardableMemory.h"
@@ -116,20 +116,20 @@ bool SkOneShotDiscardablePixelRef::onNewLockPixels(LockRec* rec) {
}
// A previous call to onUnlock may have deleted our DM, so check for that
- if (NULL == fDM) {
+ if (nullptr == fDM) {
return false;
}
if (!fDM->lock()) {
// since it failed, we delete it now, to free-up the resource
delete fDM;
- fDM = NULL;
+ fDM = nullptr;
return false;
}
SUCCESS:
rec->fPixels = fDM->data();
- rec->fColorTable = NULL;
+ rec->fColorTable = nullptr;
rec->fRowBytes = fRB;
return true;
}
@@ -164,7 +164,7 @@ bool SkResourceCacheDiscardableAllocator::allocPixelRef(SkBitmap* bitmap, SkColo
}
SkDiscardableMemory* dm = fFactory(size);
- if (NULL == dm) {
+ if (nullptr == dm) {
return false;
}
@@ -371,7 +371,7 @@ SkCachedData* SkResourceCache::newCachedData(size_t bytes) {
if (fDiscardableFactory) {
SkDiscardableMemory* dm = fDiscardableFactory(bytes);
- return dm ? new SkCachedData(bytes, dm) : NULL;
+ return dm ? new SkCachedData(bytes, dm) : nullptr;
} else {
return new SkCachedData(sk_malloc_throw(bytes), bytes);
}
@@ -396,7 +396,7 @@ void SkResourceCache::detach(Rec* rec) {
next->fPrev = prev;
}
- rec->fNext = rec->fPrev = NULL;
+ rec->fNext = rec->fPrev = nullptr;
}
void SkResourceCache::moveToHead(Rec* rec) {
@@ -421,7 +421,7 @@ void SkResourceCache::moveToHead(Rec* rec) {
void SkResourceCache::addToHead(Rec* rec) {
this->validate();
- rec->fPrev = NULL;
+ rec->fPrev = nullptr;
rec->fNext = fHead;
if (fHead) {
fHead->fPrev = rec;
@@ -440,22 +440,22 @@ void SkResourceCache::addToHead(Rec* rec) {
#ifdef SK_DEBUG
void SkResourceCache::validate() const {
- if (NULL == fHead) {
- SkASSERT(NULL == fTail);
+ if (nullptr == fHead) {
+ SkASSERT(nullptr == fTail);
SkASSERT(0 == fTotalBytesUsed);
return;
}
if (fHead == fTail) {
- SkASSERT(NULL == fHead->fPrev);
- SkASSERT(NULL == fHead->fNext);
+ SkASSERT(nullptr == fHead->fPrev);
+ SkASSERT(nullptr == fHead->fNext);
SkASSERT(fHead->bytesUsed() == fTotalBytesUsed);
return;
}
- SkASSERT(NULL == fHead->fPrev);
+ SkASSERT(nullptr == fHead->fPrev);
SkASSERT(fHead->fNext);
- SkASSERT(NULL == fTail->fNext);
+ SkASSERT(nullptr == fTail->fNext);
SkASSERT(fTail->fPrev);
size_t used = 0;
@@ -506,7 +506,7 @@ size_t SkResourceCache::getEffectiveSingleAllocationByteLimit() const {
// if we're not discardable (i.e. we are fixed-budget) then cap the single-limit
// to our budget.
- if (NULL == fDiscardableFactory) {
+ if (nullptr == fDiscardableFactory) {
if (0 == limit) {
limit = fTotalByteLimit;
} else {
@@ -527,7 +527,7 @@ void SkResourceCache::checkMessages() {
///////////////////////////////////////////////////////////////////////////////
SK_DECLARE_STATIC_MUTEX(gMutex);
-static SkResourceCache* gResourceCache = NULL;
+static SkResourceCache* gResourceCache = nullptr;
static void cleanup_gResourceCache() {
// We'll clean this up in our own tests, but disable for clients.
// Chrome seems to have funky multi-process things going on in unit tests that
@@ -542,7 +542,7 @@ static void cleanup_gResourceCache() {
static SkResourceCache* get_cache() {
// gMutex is always held when this is called, so we don't need to be fancy in here.
gMutex.assertHeld();
- if (NULL == gResourceCache) {
+ if (nullptr == gResourceCache) {
#ifdef SK_USE_DISCARDABLE_SCALEDIMAGECACHE
gResourceCache = new SkResourceCache(SkDiscardableMemory::Create);
#else
« no previous file with comments | « src/core/SkResourceCache.h ('k') | src/core/SkScalar.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698