alex/agb
1
0
Fork 0
mirror of https://github.com/italicsjenga/agb.git synced 2024-12-23 16:21:33 +11:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #326 from gwilymk/remove-pointless-set-scroll-in-affine-background

Browse source 
Remove pointless set scroll in affine background
This commit is contained in:
Gwilym Kuiper 2022-10-11 22:46:43 +01:00 committed by GitHub
parent b9d75a4fb7 982cb42b38
commit 59de262532
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 33 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

41
agb/src/display/tiled/map.rs
View file

@ -33,9 +33,6 @@ trait TiledMapPrivate: TiledMapTypes {
fn update_bg_registers(&self);
fn scroll_pos(&self) -> Vector2D<i16>;
fn set_scroll_pos(&mut self, new_pos: Vector2D<i16>);
fn bg_control_register(&self) -> MemoryMapped<u16> {
unsafe { MemoryMapped::new(0x0400_0008 + 2 * self.background_id()) }
}
@ -52,10 +49,6 @@ pub trait TiledMap: TiledMapTypes {
fn hide(&mut self);
fn commit(&mut self, vram: &mut VRamManager);
fn size(&self) -> Self::Size;
#[must_use]
fn scroll_pos(&self) -> Vector2D<i16>;
fn set_scroll_pos(&mut self, pos: Vector2D<i16>);
}
impl<T> TiledMap for T
@ -114,15 +107,6 @@ where
fn size(&self) -> T::Size {
self.map_size()
}
#[must_use]
fn scroll_pos(&self) -> Vector2D<i16> {
TiledMapPrivate::scroll_pos(self)
}
fn set_scroll_pos(&mut self, pos: Vector2D<i16>) {
TiledMapPrivate::set_scroll_pos(self, pos);
}
}
pub struct RegularMap {
@ -170,12 +154,6 @@ impl TiledMapPrivate for RegularMap {
self.x_register().set(self.scroll.x);
self.y_register().set(self.scroll.y);
}
fn scroll_pos(&self) -> Vector2D<i16> {
self.scroll
}
fn set_scroll_pos(&mut self, new_pos: Vector2D<i16>) {
self.scroll = new_pos;
}
}
impl RegularMap {
@ -230,6 +208,15 @@ impl RegularMap {
*self.tiles_dirty() = true;
}
#[must_use]
pub fn scroll_pos(&self) -> Vector2D<i16> {
self.scroll
}
pub fn set_scroll_pos(&mut self, pos: Vector2D<i16>) {
self.scroll = pos;
}
fn x_register(&self) -> MemoryMapped<i16> {
unsafe { MemoryMapped::new(0x0400_0010 + 4 * self.background_id as usize) }
}
@ -245,8 +232,6 @@ pub struct AffineMap {
priority: Priority,
size: AffineBackgroundSize,
scroll: Vector2D<i16>,
transform: AffineMatrixBackground,
tiles: Vec<u8>,
@ -282,12 +267,6 @@ impl TiledMapPrivate for AffineMap {
fn update_bg_registers(&self) {
self.bg_affine_matrix().set(self.transform);
}
fn scroll_pos(&self) -> Vector2D<i16> {
self.scroll
}
fn set_scroll_pos(&mut self, new_pos: Vector2D<i16>) {
self.scroll = new_pos;
}
}
impl AffineMap {
@ -303,8 +282,6 @@ impl AffineMap {
priority,
size,
scroll: Default::default(),
transform: Default::default(),
tiles: vec![Default::default(); size.num_tiles()],

2
examples/the-hat-chooses-the-wizard/src/level_display.rs
View file

@ -1,5 +1,5 @@
use agb::display::{
tiled::{RegularMap, TileSet, TileSetting, TiledMap, VRamManager},
tiled::{RegularMap, TileSet, TileSetting, VRamManager},
HEIGHT, WIDTH,
};