add minify flag to build script

(cherry picked from commit f0b43f7b7c)

build.ts

@@ -1,14 +1,14 @@
 #!/usr/bin/env -S deno run --allow-read
 
-if (Deno.args.length !== 0) {
-  console.error(
-    "This script takes no argument (The source file list is hardcoded) and output the result on stdout.",
-  );
-  Deno.exit(1);
-}
-
 import { TextLineStream } from "jsr:@std/streams@1.1.0";
 
+/**
+ *  CONFIG
+ */
+
+const instructionLength = 4; // in bytes
+const rawDataInstructionLengthPrefix = "U32";
+
 const asmFilePaths = [
   "src/consts/arch.asm",
   "src/consts/game.asm",
@@ -23,88 +23,162 @@ const asmFilePaths = [
 
   "src/lib/game.asm",
   "src/lib/drawing.asm",
-  "src/lib/sprite_rotation.asm",
 ];
-const reservedSpacePath = 'src/reserved_space.asm';
+const initialScreenPath = "assets/screen.ppm";
+const reservedSpacePath = "src/reserved_space.asm";
 
-const instructionLength = 4; // in bytes
-const mapWidth = 15;
-const spriteWidth = 5;
-
-const mapMatchingStrings = {
-  0x00_00_00: '0', // Empty
-  0xFF_00_00: '1', // Wall
-  0xFF_FF_00: '2', // Coin
+const mapMatchingValues = {
+  0x00_00_00: 0, // Empty
+  0xFF_00_00: 1, // Wall
+  0xFF_FF_00: 2, // Coin
 };
 
-const spritesMatchingStrings = {
-  0x00_00_00: '  0', // 0b000_000_00
-  0x00_00_FF: '  3', // 0b000_000_11
-  0x00_FF_00: ' 28', // 0b000_111_00
-  0x99_50_00: ' 68', // 0b010_001_00
-  0x77_77_77: '110', // 0b011_011_10
-  0xB3_B3_B3: '146', // 0b100_100_10
-  0xFF_00_00: '224', // 0b111_000_00
-  0xFF_00_FF: '227', // 0b111_000_11
-  0xFF_80_00: '240', // 0b111_100_00
-  0xFF_FF_00: '248', // 0b111_110_00
-  0xFF_FF_FF: '255', // 0b111_111_11
+const spritesMatchingValues = {
+  0x00_00_00: 0, // 0b000_000_00
+  0x00_00_FF: 3, // 0b000_000_11
+  0x00_FF_00: 28, // 0b000_111_00
+  0x99_50_00: 68, // 0b010_001_00
+  0x77_77_77: 110, // 0b011_011_10
+  0xB3_B3_B3: 146, // 0b100_100_10
+  0xFF_00_00: 224, // 0b111_000_00
+  0xFF_00_FF: 227, // 0b111_000_11
+  0xFF_80_00: 240, // 0b111_100_00
+  0xFF_FF_00: 248, // 0b111_110_00
+  0xFF_FF_FF: 255, // 0b111_111_11
 };
 
-const mapPath = 'assets/map.ppm';
+const mapPath = "assets/map.ppm";
 const spritesPath = {
-  'sprite_empty': 'assets/sprites/empty.ppm',
-  'sprite_unimplemented': 'assets/sprites/placeholder.ppm',
-  'sprite_wall_end': 'assets/sprites/wall_end.ppm',
-  'sprite_wall_straight': 'assets/sprites/wall_straight.ppm',
-  'sprite_wall_corner': 'assets/sprites/wall_corner.ppm',
-  'sprite_robot': 'assets/sprites/robot.ppm',
-  'sprite_ghost': 'assets/sprites/ghost.ppm',
-  'sprite_coin': 'assets/sprites/coin.ppm',
+  "sprite_empty": "assets/sprites/empty.ppm",
+  "sprite_unimplemented": "assets/sprites/placeholder.ppm",
+  "sprite_wall_end": "assets/sprites/wall_end.ppm",
+  "sprite_wall_straight": "assets/sprites/wall_straight.ppm",
+  "sprite_wall_corner": "assets/sprites/wall_corner.ppm",
+  "sprite_robot": "assets/sprites/robot.ppm",
+  "sprite_ghost": "assets/sprites/ghost.ppm",
+  "sprite_coin": "assets/sprites/coin.ppm",
 };
 
 /**
- *    START OF BUILD
+ *  END OF CONFIG
  */
 
-for (const path of asmFilePaths) {
-  const file = await Deno.open(path);
-  const readable = file.readable.pipeThrough(new TextDecoderStream())
-    .pipeThrough(new TextLineStream());
+class Minifier {
+  private buffer = "";
+  private lines: Array<string> = [];
+  stream;
 
-  console.log(`
+  constructor(writableStream: WritableStream) {
+    this.stream = new TransformStream<string>({
+      transform: (chunk, controller) => {
+        this.buffer += chunk;
+        controller.enqueue(this.processChunk());
+      },
+
+      flush: (controller) => {
+        this.buffer += "\n";
+        controller.enqueue(this.processChunk());
+      },
+    });
+
+    this.stream.readable.pipeTo(writableStream);
+  }
+
+  private processChunk(): string {
+    const newLines = this.buffer.split("\n");
+    this.buffer = newLines.pop() as string; // keep the last (potentially incomplete) line
+
+    const removeComment = (line: string) => line.replace(/\s*;.*/, "");
+    const removeLeadingWhitespaces = (line: string) => line.replace(/^\s+/, "");
+    const removeTrailingWhitespaces = (line: string) =>
+      line.replace(/\s+$/, "");
+    const removeDoubleSpaces = (line: string) => line.replaceAll(/ +/g, " ");
+    const isNotEmpty = (line: string) => line.length !== 0;
+
+    const newLinesStripped = newLines
+      .map(removeComment)
+      .map(removeLeadingWhitespaces)
+      .map(removeTrailingWhitespaces)
+      .map(removeDoubleSpaces)
+      .filter(isNotEmpty);
+
+    this.lines.push(...newLinesStripped);
+
+    return this.processLines();
+  }
+
+  private processLines() {
+    const processedLines: Array<string> = [];
+    while (this.lines.length >= instructionLength) {
+      const mergeableLines = this.lines.slice(0, instructionLength);
+      if (!mergeableLines.every((line) => line.startsWith("U8"))) {
+        processedLines.push(this.lines.shift() as string);
+        continue;
+      }
+
+      const bytesToMerge = mergeableLines.map((line) =>
+        parseInt((line.match(/U8\s(\d+)/) as RegExpMatchArray)[1])
+      );
+      let mergedBytes = 0;
+      for (let i = 0; i < bytesToMerge.length; i++) {
+        mergedBytes += bytesToMerge[i] << (instructionLength - 1 - i) * 8;
+      }
+      processedLines.push(
+        `${rawDataInstructionLengthPrefix} ${mergedBytes >>> 0}`,
+      );
+      this.lines.splice(0, instructionLength);
+    }
+
+    return processedLines.length === 0 ? "" : `${processedLines.join("\n")}\n`;
+  }
+}
+
+async function build(output: WritableStreamDefaultWriter<string>) {
+  for (const path of asmFilePaths) {
+    const file = await Deno.open(path);
+    const readable = file.readable.pipeThrough(new TextDecoderStream())
+      .pipeThrough(new TextLineStream());
+
+    output.write(`
 ;/*******************************
 ;* ${path}
 ;*******************************/
 `);
 
-  for await (const line of readable) {
-    if (!line.startsWith("include ")) {
-      console.log(line);
+    for await (const line of readable) {
+      if (!line.startsWith("include ")) {
+        output.write(`${line}\n`);
+      }
     }
   }
-}
 
+  const asmRawRepresentations = [];
 
-const asmRawRepresentations = [];
+  const mapBytes = await getDataFromPPM(mapPath);
+  asmRawRepresentations.push(
+    bytesToAsmConstU8("map", mapBytes, mapMatchingValues),
+  );
 
-const mapBytes = await getDataFromPPM(mapPath);
-asmRawRepresentations.push(bytesToAsmConstU8('map', mapBytes, mapMatchingStrings, mapWidth));
+  for (const [label, path] of Object.entries(spritesPath)) {
+    const bytes = await getDataFromPPM(path);
+    asmRawRepresentations.push(
+      bytesToAsmConstU8(label, bytes, spritesMatchingValues),
+    );
+  }
 
-for (const [label, path] of Object.entries(spritesPath)) {
-  const bytes = await getDataFromPPM(path);
-  asmRawRepresentations.push(bytesToAsmConstU8(label, bytes, spritesMatchingStrings, spriteWidth));
-}
+  output.write(asmRawRepresentations.join("\n\n"));
 
-console.log(asmRawRepresentations.join("\n\n"));
-
-console.log(`
+  output.write(`
 ;
 ; RESERVED RAM SPACE
 ;
 `);
 
-console.log(await Deno.readTextFile(reservedSpacePath))
+  const screenBytes = await getDataFromPPM(initialScreenPath);
+  output.write(bytesToAsmConstU8("screen", screenBytes, spritesMatchingValues));
+
+  output.write(await Deno.readTextFile(reservedSpacePath));
+}
 
 /**
  * Return an ASM constant (U8) representation of the given data
@@ -113,43 +187,48 @@ console.log(await Deno.readTextFile(reservedSpacePath))
  *
  * @param label The label to write before the first raw data
  * @param data An array of bytes to interprete as 24 bits values
- * @param matchingStrings An associative array with a string representation for each possible 24 bit data value
- * @param lineLength Optional: Provide a way to wrap the result in multiple lines
+ * @param matchingValues An associative array with a string representation for each possible 24 bit data value
  *
  * @returns The ASM representation of that raw data, pre- and post-fixed by the given label
  */
-function bytesToAsmConstU8(label: string, data: Uint8Array, matchingStrings: Record<number, string>, lineLength = 0): string {
-  lineLength ??= Infinity;
+function bytesToAsmConstU8(
+  label: string,
+  data: Uint8Array,
+  matchingValues: Record<number, number>,
+): string {
+  if (data.length % 3 !== 0) {
+    console.error(
+      "Length of data passed to bytesToAsmConstU8 are not a multiple of 3 (it needs to, as each resulting byte need a Red, a Green and a Blue value)",
+    );
+    Deno.exit(1);
+  }
 
-  const stringValues = [];
-  for (let i = 0; i < data.length - 2; i += 3) {
-    const pixelValue = (data[i] << 16) + (data[i+1] << 8) + (data[i+2])
-    if (!Object.hasOwn(matchingStrings, pixelValue)) {
-      console.error(`No matching value defined for "${pixelValue.toString(2)}" (which is ${label}'s ${i} pixel)`);
+  const lines = [`${label}:`];
+  for (let i = 0; i < data.length; i += 3) {
+    const pixelValue = (data[i] << 16) + (data[i + 1] << 8) + (data[i + 2]);
+    if (!Object.hasOwn(matchingValues, pixelValue)) {
+      console.error(
+        `No matching value defined for "${
+          pixelValue.toString(2)
+        }" (which is ${label}'s ${i} pixel)`,
+      );
       Deno.exit(1);
     }
 
-    stringValues.push(`U8 ${matchingStrings[pixelValue]}`);
+    lines.push(`U8 ${matchingValues[pixelValue]}`);
+  }
+  lines.push(`${label}_end:`);
+
+  const bytesNb = data.length / 3;
+  const unalignedBytesNb = bytesNb % instructionLength;
+
+  for (let i = 0; i < instructionLength - unalignedBytesNb; i++) {
+    lines.push(
+      `U8 0 ; padding to preserve ${8 * instructionLength} bits alignment`,
+    );
   }
 
-  const lines = [];
-  if (lineLength === 0) {
-    lines.push(stringValues.join("\t"));
-  } else {
-    for (let i = 0; i < stringValues.length; i += lineLength) {
-      lines.push(stringValues.slice(i, i+lineLength).join("\t"));
-    }
-  }
-
-  const paddingBytes = [];
-  for (let i = 0; i < stringValues.length % instructionLength; i++) {
-    paddingBytes.push('U8 0');
-  }
-  const paddingString = paddingBytes.length === 0 ? '' : `\n${paddingBytes.join('\t')} ; padding to preserve ${8 * instructionLength} bits alignment`
-  
-  return `${label}:
-${lines.join("\n")}
-${label}_end:${paddingString}`
+  return lines.join("\n");
 }
 
 /**
@@ -189,7 +268,7 @@ async function getDataFromPPM(path: string): Promise<Uint8Array> {
   let offset = 0;
 
   // magic number
-  while(!whitespaces.includes(content[offset])) {
+  while (!whitespaces.includes(content[offset])) {
     offset++;
   }
 
@@ -197,7 +276,7 @@ async function getDataFromPPM(path: string): Promise<Uint8Array> {
 
   // potential comment
   if (content[offset] === hashtag) {
-    while(!endOfComment.includes(content[offset])) {
+    while (!endOfComment.includes(content[offset])) {
       offset++;
     }
 
@@ -205,21 +284,21 @@ async function getDataFromPPM(path: string): Promise<Uint8Array> {
   }
 
   // image width
-  while(numbers.includes(content[offset])) {
+  while (numbers.includes(content[offset])) {
     offset++;
   }
 
   offset++; // whitespace
 
   // image height
-  while(numbers.includes(content[offset])) {
+  while (numbers.includes(content[offset])) {
     offset++;
   }
 
   offset++; // whitespace
 
   // maximum color value
-  while(numbers.includes(content[offset])) {
+  while (numbers.includes(content[offset])) {
     offset++;
   }
 
@@ -227,3 +306,51 @@ async function getDataFromPPM(path: string): Promise<Uint8Array> {
 
   return content.slice(offset, content.length);
 }
+
+function printUsage(into: (message: string) => void) {
+  into(`Usage:
+\tbuild.ts [--minify]
+\t\tMerge the hardcoded files and output the result to stdout. The "--minify" option can be used to strip the result of all comments, leading/trailing double whitespaces, empty lines and use the more compact raw data representation.
+\tbuild.ts --help
+\t\tPrint this message`);
+}
+
+/**
+ *  SCRIPT
+ */
+
+if (Deno.args.length > 1) {
+  printUsage(console.error);
+  Deno.exit(1);
+}
+
+let minify = false;
+if (Deno.args.length === 1) {
+  switch (Deno.args[0]) {
+    case "--help":
+      printUsage(console.log);
+      Deno.exit(0);
+      break;
+    case "--minify":
+      minify = true;
+      break;
+    default:
+      printUsage(console.error);
+      Deno.exit(1);
+      break;
+  }
+}
+
+const encoder = new TextEncoder();
+const stdoutWritable = new WritableStream<string>({
+  async write(chunk) {
+    await Deno.stdout.write(encoder.encode(chunk));
+  },
+});
+
+if (minify) {
+  const minifier = new Minifier(stdoutWritable);
+  await build(minifier.stream.writable.getWriter());
+} else {
+  await build(stdoutWritable.getWriter());
+}

readme.md

@@ -31,10 +31,7 @@ It also convert the PPM assets (map and sprites) into ASM raw values representat
 
 Usage example:
 ```sh
-./build.ts > /path/to/the/game/schematics/architecture/Symfony/sandbox/main.asm
-
-# script output can easily be piped to strip it off all comments, leading whitespace, double spaces and empty lines:
-./build.ts | sed -r 's/\s*;.*//' | sed -r 's/^\s+//' | sed -r 's/ +/ /' | sed -r '/^\s*$/d' > result.asm
+./build.ts --minify > /path/to/the/game/schematics/architecture/Symfony/sandbox/main.asm
 ```
 
 The script is build over Deno's API, so you'll need to have a `deno` binary in your PATH to use it that way.

src/consts/sprites.asm

@@ -2,17 +2,9 @@ pub const sprites.width = 5
 pub const sprites.height = 5 ; unused as sprite are all square
 pub const sprites.size = 25 ; height x width
 
-pub const sprites.rotation_0 = 0
-pub const sprites.rotation_90 = 1
-pub const sprites.rotation_180 = 2
-pub const sprites.rotation_270 = 3
-
 pub const sprites.empty = 0
 pub const sprites.unimplemented = 1
-pub const sprites.wall_end = 2
-pub const sprites.wall_straight = 3
-pub const sprites.wall_corner = 4
-pub const sprites.robot = 5
-pub const sprites.ghost = 6
-pub const sprites.coin = 7
-pub const sprites.max = 7 ; for lookup table
+pub const sprites.robot = 2
+pub const sprites.ghost = 3
+pub const sprites.coin = 4
+pub const sprites.max = 4 ; for lookup table

src/init.asm

@@ -17,24 +17,6 @@ lsl r2, r1, arch.instruction_increment_shift
 add r2, r2, ptr.textures_addresses
 add r3, zr, sprite_unimplemented
 store_32 [r2], r3
-; sprite_wall_end
-add r1, zr, sprites.wall_end
-lsl r2, r1, arch.instruction_increment_shift
-add r2, r2, ptr.textures_addresses
-add r3, zr, sprite_wall_end
-store_32 [r2], r3
-; sprite_wall_straight
-add r1, zr, sprites.wall_straight
-lsl r2, r1, arch.instruction_increment_shift
-add r2, r2, ptr.textures_addresses
-add r3, zr, sprite_wall_straight
-store_32 [r2], r3
-; sprite_wall_corner
-add r1, zr, sprites.wall_corner
-lsl r2, r1, arch.instruction_increment_shift
-add r2, r2, ptr.textures_addresses
-add r3, zr, sprite_wall_corner
-store_32 [r2], r3
 ; sprite_robot
 add r1, zr, sprites.robot
 lsl r2, r1, arch.instruction_increment_shift
@@ -63,7 +45,7 @@ mov r1, screen.mode_index
 screen r1, screen.mode_value
 
 mov r1, screen.offset_index
-add r2, zr, ptr.screen
+add r2, zr, screen
 screen r1, r2
 
 mov r1, screen.resolution_index

src/lib/drawing.asm

@@ -5,53 +5,6 @@ include ../consts/scene
 include ../consts/sprites
 include sprite_rotation
 
-; /* Draw the whole (fill the screen memory with the values of the sprites based on the map)
-
-; Inputs:
-
-; Outputs:
-
-; Locals:
-; 	r1: x map coord
-; 	r2: y map coord
-; 	r3: sprite rotation
-; 	r4: sprite id
-; 	r12: rotation (returned by `sprite and rotation`)
-; 	r13: sprite (returned by `get_sprite_id_and_rotation`)
-; */
-pub drawing.draw_whole_map:
-    push r1
-    push r2
-    push r3
-    push r4
-    push r12
-    push r13
-
-    mov r1, 0
-    mov r2, 0
-
-    draw_whole_map_loop:
-        call sprite_rotation.get_sprite_id_and_rotation
-        mov r3, r12
-        mov r4, r13
-        call drawing.draw_sprite_at_map_coord
-        add r1, r1, 1
-        cmp r1, map.width
-        jne draw_whole_map_loop
-        mov r1, 0
-        add r2, r2, 1
-        cmp r2, map.height
-        jne draw_whole_map_loop
-
-    pop r13
-    pop r12
-    pop r4
-    pop r3
-    pop r2
-    pop r1
-
-    ret
-
 ; /* Return the top-left pixel address of the tile at the given map coordinates
 
 ; Inputs:
@@ -77,7 +30,7 @@ get_tile_address:
 		cmp r3, sprites.height
 		jne get_tile_address_offset_per_row_loop
 
-	add r13, zr, ptr.screen
+	add r13, zr, screen
 
 	mov r3, 0
 	get_tile_address_margin_top_loop:
@@ -144,8 +97,7 @@ get_sprite_address:
 ; Inputs:
 ; 	r1: map x coord
 ; 	r2: map y coord
-; 	r3: sprite rotation
-; 	r4: sprite id
+; 	r3: sprite id
 
 ; Outputs:
 
@@ -158,22 +110,20 @@ pub drawing.draw_sprite_at_map_coord:
 	push r1
 	push r2
 	push r3
-	push r4
 	push r13
 
 	call get_tile_address
-	mov r1, r13
-	push r1
+	push r13
 	
-	mov r1, r4
+	mov r1, r3
 	call get_sprite_address
-	mov r2, r13
-	pop r1
 
+	mov r2, r13
+	pop r13
+	mov r1, r13
 	call draw_sprite
 
 	pop r13
-	pop r4
 	pop r3
 	pop r2
 	pop r1
@@ -184,18 +134,15 @@ pub drawing.draw_sprite_at_map_coord:
 ; Inputs:
 ; 	r1: screen address
 ; 	r2: sprite address
-; 	r3: rotation
 
 ; Outputs:
 
 ; Locals:
 ; 	r1: current screen pixel address
-; 	r4: x
-; 	r5: y
-; 	r6: current sprite pixel address
-; 	r7: color
-; 	r8: x increment
-; 	r9: y increment
+;		r2: current sprite pixel address
+; 	r3: x
+; 	r4: color
+;		r5: sprite last pixel address + 1
 ; */
 draw_sprite:
 	push r1
@@ -203,67 +150,25 @@ draw_sprite:
 	push r3
 	push r4
 	push r5
-	push r6
-	push r7
-	push r8
-	push r9
 	
-	mov r6, r2
+	add r5, r2, sprites.size
 
-	; rotation
-	draw_sprite_cmp_rotation_0:
-		cmp r3, sprites.rotation_0
-		jne draw_sprite_cmp_rotation_90
-		mov r8, 1
-		mov r9, 0
-		jmp draw_sprite_loop_init
-	draw_sprite_cmp_rotation_90:
-		cmp r3, sprites.rotation_90
-		jne draw_sprite_cmp_rotation_180
-		add r6, r6, sprites.size
-		sub r6, r6, sprites.width
-		sub r8, zr, sprites.width
-		mov r9, sprites.size
-		add r9, r9, 1
-		jmp draw_sprite_loop_init
-	draw_sprite_cmp_rotation_180:
-		cmp r3, sprites.rotation_180
-		jne draw_sprite_cmp_rotation_270
-		add r6, r6, sprites.size
-		sub r6, r6, 1
-		sub r8, zr, 1
-		mov r9, 0
-		jmp draw_sprite_loop_init
-	draw_sprite_cmp_rotation_270:
-		add r6, r6, sprites.width
-		sub r6, r6, 1
-		mov r8, sprites.width
-		sub r9, zr, sprites.size
-		sub r9, r9, 1
-	
-	draw_sprite_loop_init:
-	mov r4, 0
-	mov r5, 0
+	mov r3, 0
 	draw_sprite_loop:
-		load_8 r7, [r6]
-		store_8 [r1], r7
-		add r4, r4, 1
+		load_8 r4, [r2]
+		store_8 [r1], r4
+
 		add r1, r1, 1
-		add r6, r6, r8
-		cmp r4, sprites.width
+		add r2, r2, 1
+		add r3, r3, 1
+		cmp r3, sprites.width
 		jne draw_sprite_loop
-		mov r4, 0
-		add r6, r6, r9
+		mov r3, 0
 		add r1, r1, screen.width
 		sub r1, r1, sprites.width
-		add r5, r5, 1
-		cmp r5, sprites.width
+		cmp r5, r2
 		jne draw_sprite_loop
 		
-	pop r9
-	pop r8
-	pop r7
-	pop r6
 	pop r5
 	pop r4
 	pop r3

src/lib/sprite_rotation.asm

@@ -1,171 +0,0 @@
-include ../consts/offsets
-include ../consts/map
-include ../consts/sprites
-
-; /* Return the sprite id and rotation for the given map coordinates
-
-; Inputs:
-;     r1: x map coord
-;     r2: y map coord
-
-; Outputs:
-;     r12: sprite rotation
-;     r13: sprite id
-
-; Locals:
-;     r3: tile id
-;     r4: map offset
-;     r5: counter
-;     r6: left tile id
-;     r7: up tile id
-;     r8: right tile id
-;     r9: down tile id
-; */
-pub sprite_rotation.get_sprite_id_and_rotation:
-    push r1
-    push r2
-    push r3
-    push r4
-    push r5
-    push r6
-    push r7
-    push r8
-    push r9
-
-    add r4, r1, map
-    mov r5, 0
-    get_sprite_id_and_rotation_map_row_loop:
-        cmp r5, r2
-        je get_sprite_id_and_rotation_map_row_loop_end
-        add r4, r4, map.width
-        add r5, r5, 1
-        jmp get_sprite_id_and_rotation_map_row_loop
-    get_sprite_id_and_rotation_map_row_loop_end:
-
-    mov r12, sprites.rotation_0 ; default rotation
-
-    load_8 r3, [r4]
-    lsl r3, r3, 3 ; 2*instruction_size = *8 = <<3
-    add r3, r3, get_sprite_id_and_rotation_jump_table
-    jmp r3
-
-    get_sprite_id_and_rotation_jump_table:
-    ; tile_empty
-    mov r13, sprites.empty
-    jmp get_sprite_id_and_rotation_return
-    ; tiles_wall
-    jmp get_sprite_id_and_rotation_wall
-    nop
-    ; tile_coin
-    mov r13, sprites.coin
-    jmp get_sprite_id_and_rotation_return
-
-    get_sprite_id_and_rotation_wall:
-    mov r6, 0
-    mov r7, 0
-    mov r8, 0
-    mov r9, 0
-    
-    get_sprite_id_and_rotation_cmp_left:
-        cmp r1, 0
-        je get_sprite_id_and_rotation_cmp_up
-        sub r4, r4, 1
-        load_8 r6, [r4]
-        add r4, r4, 1
-    get_sprite_id_and_rotation_cmp_up:
-        cmp r2, 0
-        je get_sprite_id_and_rotation_cmp_right
-        sub r4, r4, map.width
-        load_8 r7, [r4]
-        add r4, r4, map.width
-    get_sprite_id_and_rotation_cmp_right:
-        cmp r1, map.max_x
-        je get_sprite_id_and_rotation_cmp_down
-        add r4, r4, 1
-        load_8 r8, [r4]
-        sub r4, r4, 1
-    get_sprite_id_and_rotation_cmp_down:
-        cmp r2, map.max_y
-        je get_sprite_id_and_rotation_check_connections
-        add r4, r4, map.width
-        load_8 r9, [r4]
-        sub r4, r4, map.width
-
-    get_sprite_id_and_rotation_check_connections:
-    get_sprite_id_and_rotation_straight_0:
-        cmp r7, map.tiles.wall
-        jne get_sprite_id_and_rotation_straight_90
-        cmp r9, map.tiles.wall
-        jne get_sprite_id_and_rotation_straight_90
-        mov r12, sprites.rotation_0
-        mov r13, sprites.wall_straight
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_straight_90:
-        cmp r6, map.tiles.wall
-        jne get_sprite_id_and_rotation_corner_0
-        cmp r8, map.tiles.wall
-        jne get_sprite_id_and_rotation_corner_0
-        mov r12, sprites.rotation_90
-        mov r13, sprites.wall_straight
-        jmp get_sprite_id_and_rotation_return
-
-    get_sprite_id_and_rotation_corner_0:
-        cmp r6, map.tiles.wall
-        jne get_sprite_id_and_rotation_corner_180
-        cmp r9, map.tiles.wall
-        jne get_sprite_id_and_rotation_corner_90
-        mov r12,sprites.rotation_0
-        mov r13, sprites.wall_corner
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_corner_90:
-        cmp r7, map.tiles.wall
-        jne get_sprite_id_and_rotation_end_90
-        mov r12,sprites.rotation_90
-        mov r13, sprites.wall_corner
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_corner_180:
-        cmp r8, map.tiles.wall
-        jne get_sprite_id_and_rotation_end_0
-        cmp r7, map.tiles.wall
-        jne get_sprite_id_and_rotation_corner_270
-        mov r12,sprites.rotation_180
-        mov r13, sprites.wall_corner
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_corner_270:
-        cmp r9, map.tiles.wall
-        jne get_sprite_id_and_rotation_end_270
-        mov r12,sprites.rotation_270
-        mov r13, sprites.wall_corner
-        jmp get_sprite_id_and_rotation_return
-
-    get_sprite_id_and_rotation_end_0:
-        cmp r9, map.tiles.wall
-        jne get_sprite_id_and_rotation_end_180
-        mov r12,sprites.rotation_0
-        mov r13, sprites.wall_end
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_end_90:
-        mov r12,sprites.rotation_90
-        mov r13, sprites.wall_end
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_end_180:
-        mov r12,sprites.rotation_180
-        mov r13, sprites.wall_end
-        jmp get_sprite_id_and_rotation_return
-    get_sprite_id_and_rotation_end_270:
-        mov r12,sprites.rotation_270
-        mov r13, sprites.wall_end
-        ;jmp get_sprite_id_and_rotation_return
-
-    get_sprite_id_and_rotation_return:
-    pop r9
-    pop r8
-    pop r7
-    pop r6
-    pop r5
-    pop r4
-    pop r3
-    pop r2
-    pop r1
-
-    ret

src/main.asm

@@ -2,8 +2,6 @@ include consts/screen
 
 include init
 
-call drawing.draw_whole_map
-
 ;
 ;   State initialization
 ;
@@ -22,8 +20,7 @@ store_8 [r6], r2
 add r6, r6, 1
 store_8 [r6], r5
 
-mov r3, sprites.rotation_0
-mov r4, sprites.robot
+mov r3, sprites.robot
 call drawing.draw_sprite_at_map_coord
 
 ; Ghost
@@ -41,8 +38,7 @@ store_8 [r6], r2
 add r6, r6, 1
 store_8 [r6], r5
 
-mov r3, sprites.rotation_0
-mov r4, sprites.ghost
+mov r3, sprites.ghost
 call drawing.draw_sprite_at_map_coord
 
 ;
@@ -229,15 +225,13 @@ game_loop:
     ; clear old position
     mov r1, r2
     mov r2, r3
-    mov r3, sprites.rotation_0
-    mov r4, sprites.empty
+    mov r3, sprites.empty
     call drawing.draw_sprite_at_map_coord
 
     ; draw robot on new position
     mov r1, r5
     mov r2, r6
-    mov r3, sprites.rotation_0
-    mov r4, sprites.robot
+    mov r3, sprites.robot
     call drawing.draw_sprite_at_map_coord
 
 
@@ -295,34 +289,27 @@ game_loop:
     store_8 [r1], r8
 
     ; get tile at previous position
-    push r2
     mov r1, r2
     mov r2, r3
     call game.get_tile_address_from_map_coord
-    mov r9, r13
-    pop r2
+    load_8 r13, [r13]
 
     ; draw tile at previous position
-    load_8 r1, [r9]
-    cmp r1, map.tiles.coin
+    cmp r13, map.tiles.coin
     je game_loop_ghost_movement_old_tile_coin
     ; old tile empty
-    mov r4, sprites.empty
+    mov r3, sprites.empty
     jmp game_loop_ghost_movement_draw_old_tile
     game_loop_ghost_movement_old_tile_coin:
-    mov r4, sprites.coin
+    mov r3, sprites.coin
 
     game_loop_ghost_movement_draw_old_tile:
-    mov r1, r2
-    mov r2, r3
-    mov r3, sprites.rotation_0
     call drawing.draw_sprite_at_map_coord
 
     ; draw ghost on new position
     mov r1, r7
     mov r2, r8
-    mov r3, sprites.rotation_0
-    mov r4, sprites.ghost
+    mov r3, sprites.ghost
     call drawing.draw_sprite_at_map_coord
 
     ;

src/reserved_space.asm

@@ -2,9 +2,6 @@
 pub ptr.textures_addresses:
 U32 0 ; empty
 U32 0 ; placeholder
-U32 0 ; wall_end
-U32 0 ; wall_straight
-U32 0 ; wall_corner
 U32 0 ; robot
 U32 0 ; ghost
 U32 0 ; coin
@@ -31,5 +28,3 @@ U8 0 ; x coord
 U8 0 ; y coord
 U8 0 ; direction
 U8 0 ; [padding]
-
-pub ptr.screen: