diff options
author | Rangi <35663410+Rangi42@users.noreply.github.com> | 2018-07-17 17:20:17 -0400 |
---|---|---|
committer | Rangi <35663410+Rangi42@users.noreply.github.com> | 2018-07-17 17:20:17 -0400 |
commit | 210a4b50369554e6d7b2ea9b610b2d16f2376c7a (patch) | |
tree | 14761920a2c1ac3f2f6f994ff69c6695553a3f67 /Code-cleanup.md | |
parent | 0331777b95d058b618c75bdc6078bccde9241981 (diff) |
Updated Code cleanup (markdown)
Diffstat (limited to 'Code-cleanup.md')
-rw-r--r-- | Code-cleanup.md | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/Code-cleanup.md b/Code-cleanup.md index 9feb984..5989dcf 100644 --- a/Code-cleanup.md +++ b/Code-cleanup.md @@ -6,6 +6,16 @@ These are all general issues with the pokecrystal code. Keep an eye out for oppo Reformat code to adhere to [STYLE.md](../tree/master/STYLE.md). (Converting labels to `.snake_case` is one that stands out.) +### Take advantage of up-to-date rgbds features + +As of 0.3.7, we have these features that are little-used: + +- `~` works for bytes, so we can do `~X` instead of `$ff ^ X` +- `HIGH()` and `LOW()` work for constants +- `BANK(@)` gets the current bank; `BANK("Section Name")` is also valid (some labels can thus be removed) +- `\` continues lines, like in C or Python, which could help clean up long macros (`trainer`, `npctrade`) and implement new ones (like for data/trainers/attributes.asm or data/pokemon/base_stats/\*.asm) + + ### Meaningless temporary labels Look for `UnknownText_*`, `UnknownScript_*`, `Unknown_*`, and `Function_*`. |