ref: 3648afda1694fcb4d6fe6625b14f5a3f8bc29c38
dir: /STYLE.md/
# Style Guide Some of the code will disagree with this guide. Older code is less likely to be correct. Use your best judgement. When you come across an edge case that isn't referenced in this guide, please add it. ## Comments ```asm ; Use tabs for indentation, and spaces for alignment. ; When tabs get in the way of alignment, use spaces instead. ; Comments lead with spaces after the semicolon. ; 80 char soft limit. This isn't enforced, but you should try to keep lines from being any longer. ; rgbasm doesn't have newline escapes so there's no avoiding exceeding the limit for longer macros. ; capitalization and punctuation dont matter ; but it should be consistent with the surroundings ; Space out paragraphs with two newlines. Don't put ;s on blank lines. ; Code comments should be wrapped to whatever ; is most readable rather than what's closest ; to the 80 char limit. ; Especially since most code is going to be ; under 30 chars per line, including tabs. ; Comments should go above the code they're describing, not below, and not inline. ld a, [hl] add b ld [hl], a ret ; Avoid comments on the same line as instructions. cp 10 jr c, .elsewhere ; don't do this ; If you have a larger comment and want to make ; a side note on a snippet of code: ; You can indent the comment, ; but each line should be shorter ; and spaced away from the parent comment halt nop ; To comment out code, put the ; before the tab indent. ; nop cp 10 ; jr c, .bye ret ``` ## Labels ```asm ; ROM Labels PascalCase: ; label PascalCase:: ; global label .snake_case ; local jump .PascalCase: ; an atomic chunk of code or data that's local ; don't define unused labels when a comment would do ; Labels are prefixed with lower case letters depending on location wPascalCase: ; wram sPascalCase: ; sram vPascalCase: ; vram hPascalCase: ; hram PascalCase: ; rom ; Some constants are also prefixed DEF rBGP EQU $ff47 ; hardware register ; Most other constants should be upper case DEF UPPER_CASE EQU 1 ; Long lists of constants should be aligned DEF SHORT_CONSTANT EQU 1 DEF LONGER_CONSTANT EQU 2 DEF PRETTY_LONG_CONSTANT EQU 3 DEF TINY EQU 4 DEF BUT_ONLY_RELATED_CONSTANTS EQU 5 ``` ## Directives ```asm ; meta and high-level directives should be uppercase SECTION "section", ROMX INCLUDE "filename" INCBIN "filename" MACRO my_macro nop ENDM DEF TEST EQUS "test" PURGE TEST DEF TEST EQU 2 ; data macros should be lowercase db 1 dw 2 my_macro SOME_CONSTANT ; one exception is RGB RGB 31, 31, 31 ; code macros are currently lowercase but this seems to be causing confusion with actual instructions ld b, TEST farcall DistantFunction ret ; the rest is up to you, just be consistent (prefer lowercase) set X, 1 rept 10 nop endr ``` ## Macros ```asm MACRO when_in_doubt_lowercase ; only shift if it's required or more readable ; dont db \1 shift db \1 ; do rept 10 db \1 shift endr ; do db \1, \2, \3, \4, \5 shift 5 db \1, \2, \3, \4, \5 ENDM MACRO UPPER_CASE_IS_OK_SOMETIMES ; but I can't think of any examples besides ACRONYMS ENDM ```