| Index: src/core/SkBitmap.cpp
|
| diff --git a/src/core/SkBitmap.cpp b/src/core/SkBitmap.cpp
|
| index 2fae75ae3841dd39d53c305ee83f70b8ae806fe5..38594f8f3c638f6691cb70db44e56f56f20a65e6 100644
|
| --- a/src/core/SkBitmap.cpp
|
| +++ b/src/core/SkBitmap.cpp
|
| @@ -161,7 +161,6 @@ int SkBitmap::ComputeBytesPerPixel(SkBitmap::Config config) {
|
| int bpp;
|
| switch (config) {
|
| case kNo_Config:
|
| - case kA1_Config:
|
| bpp = 0; // not applicable
|
| break;
|
| case kA8_Config:
|
| @@ -194,11 +193,6 @@ size_t SkBitmap::ComputeRowBytes(Config c, int width) {
|
| switch (c) {
|
| case kNo_Config:
|
| break;
|
| - case kA1_Config:
|
| - rowBytes.set(width);
|
| - rowBytes.add(7);
|
| - rowBytes.shiftRight(3);
|
| - break;
|
| case kA8_Config:
|
| case kIndex8_Config:
|
| rowBytes.set(width);
|
| @@ -275,7 +269,6 @@ static bool validate_alphaType(SkBitmap::Config config, SkAlphaType alphaType,
|
| case SkBitmap::kNo_Config:
|
| alphaType = kIgnore_SkAlphaType;
|
| break;
|
| - case SkBitmap::kA1_Config:
|
| case SkBitmap::kA8_Config:
|
| if (kUnpremul_SkAlphaType == alphaType) {
|
| alphaType = kPremul_SkAlphaType;
|
| @@ -291,6 +284,8 @@ static bool validate_alphaType(SkBitmap::Config config, SkAlphaType alphaType,
|
| case SkBitmap::kRGB_565_Config:
|
| alphaType = kOpaque_SkAlphaType;
|
| break;
|
| + default:
|
| + return false;
|
| }
|
| if (canonical) {
|
| *canonical = alphaType;
|
| @@ -606,8 +601,6 @@ void* SkBitmap::getAddr(int x, int y) const {
|
| case SkBitmap::kIndex8_Config:
|
| base += x;
|
| break;
|
| - case SkBitmap::kA1_Config:
|
| - base += x >> 3;
|
| break;
|
| default:
|
| SkDEBUGFAIL("Can't return addr for config");
|
| @@ -623,15 +616,6 @@ SkColor SkBitmap::getColor(int x, int y) const {
|
| SkASSERT((unsigned)y < (unsigned)this->height());
|
|
|
| switch (this->config()) {
|
| - case SkBitmap::kA1_Config: {
|
| - uint8_t* addr = this->getAddr1(x, y);
|
| - uint8_t mask = 1 << (7 - (x % 8));
|
| - if (addr[0] & mask) {
|
| - return SK_ColorBLACK;
|
| - } else {
|
| - return 0;
|
| - }
|
| - }
|
| case SkBitmap::kA8_Config: {
|
| uint8_t* addr = this->getAddr8(x, y);
|
| return SkColorSetA(0, addr[0]);
|
| @@ -654,6 +638,7 @@ SkColor SkBitmap::getColor(int x, int y) const {
|
| return SkUnPreMultiply::PMColorToColor(addr[0]);
|
| }
|
| case kNo_Config:
|
| + default:
|
| SkASSERT(false);
|
| return 0;
|
| }
|
| @@ -671,9 +656,6 @@ bool SkBitmap::ComputeIsOpaque(const SkBitmap& bm) {
|
| const int width = bm.width();
|
|
|
| switch (bm.config()) {
|
| - case SkBitmap::kA1_Config: {
|
| - // TODO
|
| - } break;
|
| case SkBitmap::kA8_Config: {
|
| unsigned a = 0xFF;
|
| for (int y = 0; y < height; ++y) {
|
| @@ -779,38 +761,6 @@ void SkBitmap::internalErase(const SkIRect& area,
|
| }
|
|
|
| switch (fConfig) {
|
| - case kA1_Config: {
|
| - uint8_t* p = this->getAddr1(area.fLeft, area.fTop);
|
| - const int left = area.fLeft >> 3;
|
| - const int right = area.fRight >> 3;
|
| -
|
| - int middle = right - left - 1;
|
| -
|
| - uint8_t leftMask = 0xFF >> (area.fLeft & 7);
|
| - uint8_t rightMask = ~(0xFF >> (area.fRight & 7));
|
| - if (left == right) {
|
| - leftMask &= rightMask;
|
| - rightMask = 0;
|
| - }
|
| -
|
| - a = (a >> 7) ? 0xFF : 0;
|
| - while (--height >= 0) {
|
| - uint8_t* startP = p;
|
| -
|
| - *p = (*p & ~leftMask) | (a & leftMask);
|
| - p++;
|
| - if (middle > 0) {
|
| - memset(p, a, middle);
|
| - p += middle;
|
| - }
|
| - if (rightMask) {
|
| - *p = (*p & ~rightMask) | (a & rightMask);
|
| - }
|
| -
|
| - p = startP + rowBytes;
|
| - }
|
| - break;
|
| - }
|
| case kA8_Config: {
|
| uint8_t* p = this->getAddr8(area.fLeft, area.fTop);
|
| while (--height >= 0) {
|
| @@ -896,7 +846,6 @@ static size_t get_sub_offset(const SkBitmap& bm, int x, int y) {
|
| break;
|
|
|
| case SkBitmap::kNo_Config:
|
| - case SkBitmap::kA1_Config:
|
| default:
|
| return SUB_OFFSET_FAILURE;
|
| }
|
| @@ -939,8 +888,6 @@ bool get_upper_left_from_offset(SkBitmap::Config config, size_t offset, size_t r
|
|
|
| case SkBitmap::kNo_Config:
|
| // Fall through.
|
| - case SkBitmap::kA1_Config:
|
| - // Fall through.
|
| default:
|
| return false;
|
| }
|
| @@ -1021,7 +968,6 @@ bool SkBitmap::canCopyTo(Config dstConfig) const {
|
| case kRGB_565_Config:
|
| case kARGB_8888_Config:
|
| break;
|
| - case kA1_Config:
|
| case kIndex8_Config:
|
| if (!sameConfigs) {
|
| return false;
|
| @@ -1032,12 +978,6 @@ bool SkBitmap::canCopyTo(Config dstConfig) const {
|
| default:
|
| return false;
|
| }
|
| -
|
| - // do not copy src if srcConfig == kA1_Config while dstConfig != kA1_Config
|
| - if (this->config() == kA1_Config && !sameConfigs) {
|
| - return false;
|
| - }
|
| -
|
| return true;
|
| }
|
|
|
| @@ -1681,7 +1621,7 @@ void SkBitmap::validate() const {
|
| void SkBitmap::toString(SkString* str) const {
|
|
|
| static const char* gConfigNames[kConfigCount] = {
|
| - "NONE", "A1", "A8", "INDEX8", "565", "4444", "8888"
|
| + "NONE", "A8", "INDEX8", "565", "4444", "8888"
|
| };
|
|
|
| str->appendf("bitmap: ((%d, %d) %s", this->width(), this->height(),
|
|
|