Use assertions to catch the GetName bug, and use a NAME_BUFFER_LENGTH constant

constants/text_constants.asm

@@ -1,5 +1,6 @@
 NAME_LENGTH EQU 11
 ITEM_NAME_LENGTH EQU 13
+NAME_BUFFER_LENGTH EQU 20
 
 ; PrintNumber
 	const_def 5

engine/movie/oak_speech/oak_speech2.asm

@@ -210,7 +210,7 @@ GetDefaultName:
 	ld h, d
 	ld l, e
 	ld de, wcd6d
-	ld bc, $14
+	ld bc, NAME_BUFFER_LENGTH
 	jp CopyData
 
 INCLUDE "data/player_names_list.asm"

home/names2.asm

@@ -20,6 +20,12 @@ GetName::
 
 	; TM names are separate from item names.
 	; BUG: This applies to all names instead of just items.
+	assert NUM_POKEMON_INDEXES < HM01, \
+		"A bug in GetName will get TM/HM names for Pokémon above ${x:HM01}."
+	assert NUM_ATTACKS < HM01, \
+		"A bug in GetName will get TM/HM names for moves above ${x:HM01}."
+	assert NUM_TRAINERS < HM01, \
+		"A bug in GetName will get TM/HM names for trainers above ${x:HM01}."
 	cp HM01
 	jp nc, GetMachineName
 
@@ -28,10 +34,10 @@ GetName::
 	push hl
 	push bc
 	push de
-	ld a, [wNameListType]    ;List3759_entrySelector
+	ld a, [wNameListType]
 	dec a
 	jr nz, .otherEntries
-	;1 = MON_NAMES
+	; 1 = MONSTER_NAME
 	call GetMonName
 	ld hl, NAME_LENGTH
 	add hl, de
@@ -39,11 +45,11 @@ GetName::
 	ld d, h
 	jr .gotPtr
 .otherEntries
-	;2-7 = OTHER ENTRIES
+	; 2-7 = other names
 	ld a, [wPredefBank]
 	ldh [hLoadedROMBank], a
 	ld [MBC1RomBank], a
-	ld a, [wNameListType]    ;VariousNames' entryID
+	ld a, [wNameListType]
 	dec a
 	add a
 	ld d, 0
@@ -62,8 +68,8 @@ GetName::
 	ldh a, [hSwapTemp + 1]
 	ld l, a
 	ld a, [wd0b5]
-	ld b, a
-	ld c, 0
+	ld b, a ; wanted entry
+	ld c, 0 ; entry counter
 .nextName
 	ld d, h
 	ld e, l
@@ -71,14 +77,14 @@ GetName::
 	ld a, [hli]
 	cp "@"
 	jr nz, .nextChar
-	inc c           ;entry counter
-	ld a, b          ;wanted entry
+	inc c
+	ld a, b
 	cp c
 	jr nz, .nextName
 	ld h, d
 	ld l, e
 	ld de, wcd6d
-	ld bc, $14
+	ld bc, NAME_BUFFER_LENGTH
 	call CopyData
 .gotPtr
 	ld a, e

wram.asm

@@ -1173,7 +1173,11 @@ wNumMovesMinusOne::
 
 UNION
 
-wcd6d:: ds 4 ; buffer for various data
+wcd6d:: ds NAME_BUFFER_LENGTH ; buffer for various data
+
+NEXTU
+
+	ds 4
 
 wStatusScreenCurrentPP::
 ; temp variable used to print a move's current PP on the status screen