Compare commits

..

No commits in common. "905c4448d08853bc4776f90dfe844ed13da34eee" and "5aac04faf59e45bda912e297a7c0a40e59bf00d9" have entirely different histories.

1 changed files with 32 additions and 60 deletions

View File

@ -444,7 +444,7 @@ pub const Ppu = struct {
// Sprite Palette starts at 0x0500_0200
if (pal_id != 0) {
const bgr555 = self.palette.read(u16, 0x200 + pal_id * 2);
drawSpritePixel(self.bld.cnt, &self.scanline, @bitCast(Attr0, sprite.attr0), global_x, bgr555);
drawSpritePixel(self.bld.cnt, &self.scanline, global_x, bgr555);
}
}
}
@ -494,7 +494,7 @@ pub const Ppu = struct {
// Sprite Palette starts at 0x0500_0200
if (pal_id != 0) {
const bgr555 = self.palette.read(u16, 0x200 + pal_id * 2);
drawSpritePixel(self.bld.cnt, &self.scanline, sprite.attr0, x, bgr555);
drawSpritePixel(self.bld.cnt, &self.scanline, x, bgr555);
}
}
}
@ -725,25 +725,21 @@ pub const Ppu = struct {
fn getBgr555(self: *Self, maybe_top: Scanline.Pixel, maybe_btm: Scanline.Pixel) u16 {
return switch (self.bld.cnt.mode.read()) {
0b00 => switch (maybe_top) {
.set, .obj_set => |top| top,
.set => |top| top,
else => self.palette.backdrop(),
},
0b01 => switch (maybe_top) {
.set, .obj_set => |top| switch (maybe_btm) {
.set, .obj_set => |btm| alphaBlend(top, btm, self.bld.alpha), // ALPHA_BLEND
.set => |top| switch (maybe_btm) {
.set => |btm| alphaBlend(top, btm, self.bld.alpha), // ALPHA_BLEND
else => top,
},
else => switch (maybe_btm) {
.set, .obj_set => |btm| btm,
.set => |btm| btm,
else => self.palette.backdrop(),
},
},
0b10 => switch (maybe_btm) {
.set, .obj_set => |btm| blk: {
// If there's a top pixel + this btm pixel came from a sprite
// don't display top pixel + don't blend btm pixel
if (maybe_btm == .obj_set and maybe_top.isSet()) break :blk btm;
.set => |btm| blk: {
// BLD_WHITE
const evy: u16 = self.bld.y.evy.read();
@ -758,16 +754,12 @@ pub const Ppu = struct {
break :blk (bld_b << 10) | (bld_g << 5) | bld_r;
},
else => switch (maybe_top) {
.set, .obj_set => |top| top,
.set => |top| top,
else => self.palette.backdrop(),
},
},
0b11 => switch (maybe_btm) {
.set, .obj_set => |btm| blk: {
// If there's a top pixel + this btm pixel came from a sprite
// don't display top pixel + don't blend btm pixel
if (maybe_btm == .obj_set and maybe_top.isSet()) break :blk btm;
.set => |btm| blk: {
// BLD_BLACK
const evy: u16 = self.bld.y.evy.read();
@ -782,7 +774,7 @@ pub const Ppu = struct {
break :blk (bld_b << 10) | (bld_g << 5) | bld_r;
},
else => switch (maybe_top) {
.set, .obj_set => |top| top,
.set => |top| top,
else => self.palette.backdrop(),
},
},
@ -804,7 +796,7 @@ pub const Ppu = struct {
const is_top_layer = (top_layer >> layer) & 1 == 1;
if (is_top_layer) {
self.scanline.top()[i] = Scanline.Pixel.from(.Background, bgr555);
self.scanline.top()[i] = Scanline.Pixel.from(bgr555);
return;
}
@ -813,7 +805,7 @@ pub const Ppu = struct {
const is_btm_layer = (btm_layer >> layer) & 1 == 1;
if (is_btm_layer) {
self.scanline.btm()[i] = Scanline.Pixel.from(.Background, bgr555);
self.scanline.btm()[i] = Scanline.Pixel.from(bgr555);
return;
}
@ -834,14 +826,14 @@ pub const Ppu = struct {
const is_top_layer = (top_layer >> layer) & 1 == 1;
if (is_top_layer) {
self.scanline.btm()[i] = Scanline.Pixel.from(.Background, bgr555); // this is intentional
self.scanline.btm()[i] = Scanline.Pixel.from(bgr555); // this is intentional
return;
}
},
}
// If we aren't blending here at all, just add the pixel to the top layer
self.scanline.top()[i] = Scanline.Pixel.from(.Background, bgr555);
self.scanline.top()[i] = Scanline.Pixel.from(bgr555);
}
const WindowBounds = enum { win0, win1, out };
@ -867,7 +859,7 @@ pub const Ppu = struct {
fn shouldDrawBackground(self: *Self, comptime layer: u2, bounds: ?WindowBounds, i: usize) bool {
switch (self.bld.cnt.mode.read()) {
0b00 => if (self.scanline.top()[i].isSet()) return false, // pass through
0b00 => if (self.scanline.top()[i] == .set) return false, // pass through
0b01 => blk: {
// BLD_ALPHA
@ -876,7 +868,7 @@ pub const Ppu = struct {
const is_btm_layer = (btm_layer >> layer) & 1 == 1;
if (is_btm_layer) {
if (self.scanline.btm()[i].isSet()) return false;
if (self.scanline.btm()[i] == .set) return false;
// In some previous iteration we have determined that an opaque pixel was drawn at this position
// therefore there's no reason to draw anything here
@ -900,7 +892,7 @@ pub const Ppu = struct {
break :blk;
}
if (self.scanline.top()[i].isSet()) return false;
if (self.scanline.top()[i] == .set) return false;
},
0b10, 0b11 => {
// BLD_WHITE and BLD_BLACK
@ -908,9 +900,7 @@ pub const Ppu = struct {
// we want to treat the bottom layer the same as the top (despite it being repurposed)
// so we should apply the same logic to the bottom layer
if (self.scanline.top()[i].isSet()) return false;
// If the bottom pixel comes rom a sprite, draw the pixel anyways
if (self.scanline.top()[i] == .set) return false;
if (self.scanline.btm()[i] == .set) return false;
},
}
@ -1419,8 +1409,10 @@ fn alphaBlend(top: u16, btm: u16, bldalpha: io.BldAlpha) u16 {
}
fn shouldDrawSprite(bldcnt: io.BldCnt, scanline: *Scanline, x: u9) bool {
if (scanline.top()[x] == .set) return false;
switch (bldcnt.mode.read()) {
0b00 => if (scanline.top()[x].isSet()) return false,
0b00 => if (scanline.top()[x] == .set) return false, // pass through
0b01 => {
// BLD_ALPHA
@ -1429,26 +1421,20 @@ fn shouldDrawSprite(bldcnt: io.BldCnt, scanline: *Scanline, x: u9) bool {
const btm_layers = bldcnt.layer_b.read();
const is_btm_layer = (btm_layers >> 4) & 1 == 1;
if (is_btm_layer and scanline.btm()[x].isSet()) return false;
if (is_btm_layer and scanline.btm()[x] == .set) return false;
if (scanline.top()[x].isSet()) return false;
if (scanline.top()[x] == .set) return false;
},
0b10, 0b11 => {
if (scanline.top()[x].isSet()) return false;
if (scanline.btm()[x].isSet()) return false;
if (scanline.top()[x] == .set) return false;
if (scanline.btm()[x] == .set) return false;
},
}
return true;
}
fn drawSpritePixel(bldcnt: io.BldCnt, scanline: *Scanline, attr0: Attr0, x: u9, bgr555: u16) void {
if (attr0.mode.read() == 1) {
// TODO: Force Alpha Blend in all moes?
scanline.top()[x] = Scanline.Pixel.from(.Sprite, bgr555);
return;
}
fn drawSpritePixel(bldcnt: io.BldCnt, scanline: *Scanline, x: u9, bgr555: u16) void {
switch (bldcnt.mode.read()) {
0b00 => {}, // pass through
0b01 => {
@ -1457,7 +1443,7 @@ fn drawSpritePixel(bldcnt: io.BldCnt, scanline: *Scanline, attr0: Attr0, x: u9,
const is_top_layer = (top_layers >> 4) & 1 == 1;
if (is_top_layer) {
scanline.top()[x] = Scanline.Pixel.from(.Sprite, bgr555);
scanline.top()[x] = Scanline.Pixel.from(bgr555);
return;
}
@ -1465,7 +1451,7 @@ fn drawSpritePixel(bldcnt: io.BldCnt, scanline: *Scanline, attr0: Attr0, x: u9,
const is_btm_layer = (btm_layers >> 4) & 1 == 1;
if (is_btm_layer) {
scanline.btm()[x] = Scanline.Pixel.from(.Sprite, bgr555);
scanline.btm()[x] = Scanline.Pixel.from(bgr555);
return;
}
@ -1482,39 +1468,25 @@ fn drawSpritePixel(bldcnt: io.BldCnt, scanline: *Scanline, attr0: Attr0, x: u9,
const is_top_layer = (top_layers >> 4) & 1 == 1;
if (is_top_layer) {
scanline.btm()[x] = Scanline.Pixel.from(.Sprite, bgr555); // This is intentional
scanline.btm()[x] = Scanline.Pixel.from(bgr555); // This is intentional
return;
}
},
}
scanline.top()[x] = Scanline.Pixel.from(.Sprite, bgr555);
scanline.top()[x] = Scanline.Pixel.from(bgr555);
}
const Scanline = struct {
const Self = @This();
const Pixel = union(enum) {
// TODO: Rename
const Layer = enum { Background, Sprite };
set: u16,
obj_set: u16,
unset: void,
hidden: void,
fn from(comptime layer: Layer, bgr555: u16) Pixel {
return switch (layer) {
.Background => .{ .set = bgr555 },
.Sprite => .{ .obj_set = bgr555 },
};
}
pub fn isSet(self: @This()) bool {
return switch (self) {
.set, .obj_set => true,
.unset, .hidden => false,
};
fn from(bgr555: u16) Pixel {
return .{ .set = bgr555 };
}
};