summaryrefslogtreecommitdiff
path: root/Code-cleanup.md
diff options
context:
space:
mode:
authorRangi <35663410+Rangi42@users.noreply.github.com>2018-07-17 17:20:17 -0400
committerRangi <35663410+Rangi42@users.noreply.github.com>2018-07-17 17:20:17 -0400
commit210a4b50369554e6d7b2ea9b610b2d16f2376c7a (patch)
tree14761920a2c1ac3f2f6f994ff69c6695553a3f67 /Code-cleanup.md
parent0331777b95d058b618c75bdc6078bccde9241981 (diff)
Updated Code cleanup (markdown)
Diffstat (limited to 'Code-cleanup.md')
-rw-r--r--Code-cleanup.md10
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_*`.