diff options
author | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-04-23 19:19:44 +0000 |
---|---|---|
committer | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-04-23 19:19:44 +0000 |
commit | f117781362ecf673f43f93918781853690f0e145 (patch) | |
tree | f6be20d37ac41564b878b11a5cd86a64d76714c5 /src | |
parent | edbd6ae92e8bc2431ec2f97d6e0d02c9c74c3e7f (diff) |
fix size_t/int warnings
BUG=skia:
R=mtklein@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/247753003
git-svn-id: http://skia.googlecode.com/svn/trunk@14332 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src')
-rw-r--r-- | src/core/SkReadBuffer.cpp | 4 | ||||
-rw-r--r-- | src/core/SkRegion.cpp | 4 | ||||
-rw-r--r-- | src/core/SkRegionPriv.h | 4 | ||||
-rw-r--r-- | src/core/SkScalerContext.cpp | 5 | ||||
-rw-r--r-- | src/core/SkStream.cpp | 16 | ||||
-rw-r--r-- | src/core/SkValidatingReadBuffer.cpp | 4 | ||||
-rw-r--r-- | src/core/SkWriter32.cpp | 2 | ||||
-rw-r--r-- | src/sfnt/SkOTUtils.cpp | 6 |
8 files changed, 22 insertions, 23 deletions
diff --git a/src/core/SkReadBuffer.cpp b/src/core/SkReadBuffer.cpp index b60dee3ba6..b4bc87529a 100644 --- a/src/core/SkReadBuffer.cpp +++ b/src/core/SkReadBuffer.cpp @@ -321,10 +321,10 @@ SkFlattenable* SkReadBuffer::readFlattenable(SkFlattenable::Type ft) { SkFlattenable* obj = NULL; uint32_t sizeRecorded = fReader.readU32(); if (factory) { - uint32_t offset = fReader.offset(); + size_t offset = fReader.offset(); obj = (*factory)(*this); // check that we read the amount we expected - uint32_t sizeRead = fReader.offset() - offset; + size_t sizeRead = fReader.offset() - offset; if (sizeRecorded != sizeRead) { // we could try to fix up the offset... sk_throw(); diff --git a/src/core/SkRegion.cpp b/src/core/SkRegion.cpp index baedf2aea8..98670b6b52 100644 --- a/src/core/SkRegion.cpp +++ b/src/core/SkRegion.cpp @@ -796,7 +796,7 @@ public: fTop = (SkRegion::RunType)(bottom); // just update our bottom } else { start[-2] = (SkRegion::RunType)(bottom); - start[-1] = len >> 1; + start[-1] = SkToS32(len >> 1); fPrevDst = start; fPrevLen = len; } @@ -1212,7 +1212,7 @@ static void compute_bounds(const SkRegion::RunType runs[], const SkRegion::RunType* prev = runs; runs = skip_intervals_slow(runs); - int intervals = (runs - prev) >> 1; + int intervals = SkToInt((runs - prev) >> 1); SkASSERT(prev[-1] == intervals); intervalCount += intervals; diff --git a/src/core/SkRegionPriv.h b/src/core/SkRegionPriv.h index f299f3a9d6..c8f000df35 100644 --- a/src/core/SkRegionPriv.h +++ b/src/core/SkRegionPriv.h @@ -29,7 +29,7 @@ static int compute_intervalcount(const SkRegion::RunType runs[]) { SkASSERT(curr[1] < SkRegion::kRunTypeSentinel); curr += 2; } - return (curr - runs) >> 1; + return SkToInt((curr - runs) >> 1); } #endif @@ -213,7 +213,7 @@ public: #ifdef SK_DEBUG // +1 to skip the last Y-sentinel - int runCount = runs - this->writable_runs() + 1; + int runCount = SkToInt(runs - this->writable_runs() + 1); SkASSERT(runCount == fRunCount); #endif diff --git a/src/core/SkScalerContext.cpp b/src/core/SkScalerContext.cpp index d0d24eebe8..fee1ff7908 100644 --- a/src/core/SkScalerContext.cpp +++ b/src/core/SkScalerContext.cpp @@ -548,8 +548,9 @@ static void packA8ToA1(const SkMask& mask, const uint8_t* src, size_t srcRB) { const int dstPad = mask.fRowBytes - SkAlign8(width)/8; SkASSERT(dstPad >= 0); - const int srcPad = srcRB - width; - SkASSERT(srcPad >= 0); + SkASSERT(width >= 0); + SkASSERT(srcRB >= (size_t)width); + const size_t srcPad = srcRB - width; for (int y = 0; y < height; ++y) { for (int i = 0; i < octs; ++i) { diff --git a/src/core/SkStream.cpp b/src/core/SkStream.cpp index c60f2a738e..ebaac9ab15 100644 --- a/src/core/SkStream.cpp +++ b/src/core/SkStream.cpp @@ -161,7 +161,7 @@ bool SkWStream::writePackedUInt(size_t value) { memcpy(&data[1], &value16, 2); len = 3; } else { - uint32_t value32 = value; + uint32_t value32 = SkToU32(value); data[0] = SK_BYTE_SENTINEL_FOR_U32; memcpy(&data[1], &value32, 4); len = 5; @@ -189,7 +189,7 @@ bool SkWStream::writeStream(SkStream* stream, size_t length) { bool SkWStream::writeData(const SkData* data) { if (data) { - this->write32(data->size()); + this->write32(SkToU32(data->size())); this->write(data->data(), data->size()); } else { this->write32(0); @@ -481,11 +481,9 @@ SkMemoryWStream::SkMemoryWStream(void* buffer, size_t size) { } -bool SkMemoryWStream::write(const void* buffer, size_t size) -{ - size = SkMin32(size, fMaxLength - fBytesWritten); - if (size > 0) - { +bool SkMemoryWStream::write(const void* buffer, size_t size) { + size = SkTMin(size, fMaxLength - fBytesWritten); + if (size > 0) { memcpy(fBuffer + fBytesWritten, buffer, size); fBytesWritten += size; return true; @@ -558,7 +556,7 @@ bool SkDynamicMemoryWStream::write(const void* buffer, size_t count) size_t size; if (fTail != NULL && fTail->avail() > 0) { - size = SkMin32(fTail->avail(), count); + size = SkTMin(fTail->avail(), count); buffer = fTail->append(buffer, size); SkASSERT(count >= size); count -= size; @@ -566,7 +564,7 @@ bool SkDynamicMemoryWStream::write(const void* buffer, size_t count) return true; } - size = SkMax32(count, SkDynamicMemoryWStream_MinBlockSize); + size = SkTMax<size_t>(count, SkDynamicMemoryWStream_MinBlockSize); Block* block = (Block*)sk_malloc_throw(sizeof(Block) + size); block->init(size); block->append(buffer, count); diff --git a/src/core/SkValidatingReadBuffer.cpp b/src/core/SkValidatingReadBuffer.cpp index 8db2c68b9a..95bf83c847 100644 --- a/src/core/SkValidatingReadBuffer.cpp +++ b/src/core/SkValidatingReadBuffer.cpp @@ -256,10 +256,10 @@ SkFlattenable* SkValidatingReadBuffer::readFlattenable(SkFlattenable::Type type) SkFlattenable* obj = NULL; uint32_t sizeRecorded = this->readUInt(); if (factory) { - uint32_t offset = fReader.offset(); + size_t offset = fReader.offset(); obj = (*factory)(*this); // check that we read the amount we expected - uint32_t sizeRead = fReader.offset() - offset; + size_t sizeRead = fReader.offset() - offset; this->validate(sizeRecorded == sizeRead); if (fError) { // we could try to fix up the offset... diff --git a/src/core/SkWriter32.cpp b/src/core/SkWriter32.cpp index c7bfd92d56..3397c37360 100644 --- a/src/core/SkWriter32.cpp +++ b/src/core/SkWriter32.cpp @@ -47,7 +47,7 @@ void SkWriter32::writeString(const char str[], size_t len) { // [ 4 byte len ] [ str ... ] [1 - 4 \0s] uint32_t* ptr = this->reservePad(sizeof(uint32_t) + len + 1); - *ptr = len; + *ptr = SkToU32(len); char* chars = (char*)(ptr + 1); memcpy(chars, str, len); chars[len] = '\0'; diff --git a/src/sfnt/SkOTUtils.cpp b/src/sfnt/SkOTUtils.cpp index 004a888310..e76d1da08c 100644 --- a/src/sfnt/SkOTUtils.cpp +++ b/src/sfnt/SkOTUtils.cpp @@ -103,7 +103,7 @@ SkData* SkOTUtils::RenameFont(SkStream* fontData, const char* fontName, int font for (; currentEntry < endEntry; ++currentEntry) { uint32_t oldOffset = SkEndian_SwapBE32(currentEntry->offset); if (oldOffset > oldNameTableOffset) { - currentEntry->offset = SkEndian_SwapBE32(oldOffset - oldNameTablePhysicalSize); + currentEntry->offset = SkEndian_SwapBE32(SkToU32(oldOffset - oldNameTablePhysicalSize)); } if (SkOTTableHead::TAG == currentEntry->tag) { headTableEntry = currentEntry; @@ -112,8 +112,8 @@ SkData* SkOTUtils::RenameFont(SkStream* fontData, const char* fontName, int font // Make the table directory entry point to the new 'name' table. SkSFNTHeader::TableDirectoryEntry* nameTableEntry = reinterpret_cast<SkSFNTHeader::TableDirectoryEntry*>(data + sizeof(SkSFNTHeader)) + tableIndex; - nameTableEntry->logicalLength = SkEndian_SwapBE32(nameTableLogicalSize); - nameTableEntry->offset = SkEndian_SwapBE32(originalDataSize); + nameTableEntry->logicalLength = SkEndian_SwapBE32(SkToU32(nameTableLogicalSize)); + nameTableEntry->offset = SkEndian_SwapBE32(SkToU32(originalDataSize)); // Write the new 'name' table after the original font data. SkOTTableName* nameTable = reinterpret_cast<SkOTTableName*>(data + originalDataSize); |