High-level mages get told they can learn 1 more spell when there are no more spells left to learn

[magnate]

Confirmed - there are 58 spells in the 9 books, and they can learn 59 with maxed INT. Problem is somewhere in calc_spells (player/calcs.c) but I can't see it yet. Something to do with num_allowed or PY_MAX_SPELLS etc.

[noz]

Spell 7 (used to be Detect Objects) no longer exists, so the spell level is zero. When the number of available spells is counted, the condition is if spell_level <= player_level, then count it. However, this will end up counting an extra 1 for spell 7.

Fix:

Index: src/player/calcs.c
===================================================================
--- src/player/calcs.c	(revision 1401)
+++ src/player/calcs.c	(working copy)
@@ -196,8 +196,8 @@
 		/* Get the spell */
 		s_ptr = &mp_ptr->info[j];
 
-		/* Skip spells we cannot remember */
-		if (s_ptr->slevel > p_ptr->lev) continue;
+		/* Skip spells we cannot remember or don't exist */
+		if (s_ptr->slevel > p_ptr->lev || s_ptr->slevel == 0) continue;
 
 		/* Skip spells we already know */
 		if (p_ptr->spell_flags[j] & PY_SPELL_LEARNED)

[magnate]

[743b76d] (SVN r1403). Thanks to Peter Denison for the patch.

[magnate]

Now a problem for priests since merging detect traps/doors/stairs. See ​http://angband.oook.cz/forum/showthread.php?t=4195.

[magnate]

I can confirm that priests have one spell left to learn after learning all spells in all books, but can see no simple fix like previously. According to p_class.txt they have 57 to learn, but the game says 58.

[magnate]

Fixed by noz in [rc0cea93].

[magnate]

In master as of [r72e6029], Sun 22 May 2011.