all repos — mgba @ c24f31d2a740eb5970707e9d94532f42a1379a79

mGBA Game Boy Advance Emulator

DS GX: Ensure palBase is non-null
Vicki Pfau vi@endrift.com
Thu, 06 Apr 2017 15:29:01 -0700
commit

c24f31d2a740eb5970707e9d94532f42a1379a79

parent

0bdf30f650b219c7f17b9a015f63c5cb681a2399

1 files changed, 14 insertions(+), 10 deletions(-)

jump to
M src/ds/gx/software.csrc/ds/gx/software.c

@@ -94,6 +94,9 @@ static color_t _lookupColor(struct DSGXSoftwareRenderer* renderer, struct DSGXSoftwareEndpoint* ep, struct DSGXSoftwarePolygon* poly) {

if (!poly->texBase && poly->texFormat) { return 0; } + if (!poly->palBase && poly->texFormat && poly->texFormat != 7) { + return 0; + } // TODO: Optimize uint16_t texel;

@@ -450,22 +453,23 @@ poly->texFormat = DSGXTexParamsGetFormat(poly->texParams);

poly->blendFormat = DSGXPolygonAttrsGetMode(poly->polyParams); poly->texW = 8 << DSGXTexParamsGetSSize(poly->texParams); poly->texH = 8 << DSGXTexParamsGetTSize(poly->texParams); - if (!renderer->tex[DSGXTexParamsGetVRAMBase(poly->texParams) >> VRAM_BLOCK_OFFSET]) { - poly->texBase = NULL; - poly->palBase = NULL; - } else { + poly->texBase = NULL; + poly->palBase = NULL; + if (renderer->tex[DSGXTexParamsGetVRAMBase(poly->texParams) >> VRAM_BLOCK_OFFSET]) { switch (poly->texFormat) { case 0: - poly->texBase = NULL; - poly->palBase = NULL; break; case 2: - poly->texBase = &renderer->tex[DSGXTexParamsGetVRAMBase(poly->poly->texParams) >> VRAM_BLOCK_OFFSET][(DSGXTexParamsGetVRAMBase(poly->poly->texParams) << 2) & 0xFFFF]; - poly->palBase = &renderer->texPal[poly->poly->palBase >> 11][(poly->poly->palBase << 2) & 0x1FFF]; + if (renderer->texPal[poly->poly->palBase >> 11]) { + poly->texBase = &renderer->tex[DSGXTexParamsGetVRAMBase(poly->poly->texParams) >> VRAM_BLOCK_OFFSET][(DSGXTexParamsGetVRAMBase(poly->poly->texParams) << 2) & 0xFFFF]; + poly->palBase = &renderer->texPal[poly->poly->palBase >> 11][(poly->poly->palBase << 2) & 0x1FFF]; + } break; default: - poly->texBase = &renderer->tex[DSGXTexParamsGetVRAMBase(poly->poly->texParams) >> VRAM_BLOCK_OFFSET][(DSGXTexParamsGetVRAMBase(poly->poly->texParams) << 2) & 0xFFFF]; - poly->palBase = &renderer->texPal[poly->poly->palBase >> 10][(poly->poly->palBase << 3) & 0x1FFF]; + if (renderer->texPal[poly->poly->palBase >> 10]) { + poly->texBase = &renderer->tex[DSGXTexParamsGetVRAMBase(poly->poly->texParams) >> VRAM_BLOCK_OFFSET][(DSGXTexParamsGetVRAMBase(poly->poly->texParams) << 2) & 0xFFFF]; + poly->palBase = &renderer->texPal[poly->poly->palBase >> 10][(poly->poly->palBase << 3) & 0x1FFF]; + } break; } }