zladx/LADX-Disassembly

Find what the unused entities were supposed to do

Open

#142 geöffnet am 12. Nov. 2019

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Assembly (89 Forks)github user discovery
help wanted

Repository-Metriken

Stars
 (886 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Several unknown entities are unused throughout the code.

These are mostly visible in the entities handlers table: all the entities labeled EntityXXXHandler are not used in the final game–but point to actual code.

For instance these two ones (but there are others)

https://github.com/zladx/LADX-Disassembly/blob/7f2c25ba394b75748e7ceb55677bc9d6b5115c70/src/code/entities/_handlers_table.asm#L120-L121

It could be interesting to understand what where these entities, and why they were eventually left out of the final game.

Resources

Maybe some leftover elements mentioned on The Cutting Room Floor could provide some hints.

Contributor Guide

Research-Richtung
Untersuchen Sie die nicht verwendeten Entitätshandler in handlers table.asm und vergleichen Sie sie mit bekannten Spielinhalten. Überprüfen Sie The Cutting Room Floor auf nicht verwendete Elemente und gleichen Sie die Referenzen mit dem Code ab, um das beabsichtigte Verhalten zu ermitteln.
Tech Stack
Keine
Domain
reverse engineeringgame dev
Issue Type
Recherche
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Assembly languageUnderstanding of Game Boy disassembly
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60