<div dir="ltr"><div>I guess there is a small chance of this bufix breaking a game, if it relied on spell costs being ignored for a hero's random spell list. Unlikely, but possible.</div><div><br></div><div>I'm tempted to just leave this as a simple fix, and only add a backcompat bit if somebody finds a game that is actually broken by this. (And I have zero problem with adding a line in the upgrade sub that does if gamename = "somegame" then flip fixbit)</div><div><br></div><div>What are your thoughts?</div><div><br></div><div>I see the attack costs megabug suggests a shared fix bit "Correct handling of attack costs" which I could create now-- though I think doing that for this fix means I am committing to fix all four of the bugs mentioned under that fixbit also before the next stable release (which is not a bad thing I guess?)<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 8, 2022 at 4:03 PM James Paige <<a href="mailto:Bob@hamsterrepublic.com">Bob@hamsterrepublic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Oh, I missed that this one had already been split out into a separate bug <a href="https://github.com/ohrrpgce/ohrrpgce/issues/394" target="_blank">https://github.com/ohrrpgce/ohrrpgce/issues/394</a></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 8, 2022 at 4:01 PM <<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">james<br>
2022-02-08 13:01:45 -0800 (Tue, 08 Feb 2022)<br>
103<br>
Fix random spell lists to check atkallowed()<br>
<br>
resolves one tiny little part of the attack costs megabug<br>
---<br>
U   wip/bmod.rbas<br>
U   wip/bmodsubs.bas<br>
U   wip/<a href="http://bmodsubs.bi" rel="noreferrer" target="_blank">bmodsubs.bi</a><br>
<br>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div>
</blockquote></div>