<div dir="ltr"><div>This was a big bug (although I read it only affected attacks set to "Automatically choose target"), so I think it's likely that there are games which are significantly affected. I'd rather have a backcompat bit for it.<br></div><div></div><div><br></div><div>Actually, I wasn't suggesting that shared backcompat bit for this bugfix, because I didn't even know about this bug when I wrote that first comment. I still think grouping together bugfixes into one or a few backcompat bits is good, but if you fix some now and some later it can just be split into multiple bits... or if there's a bit of a delay before all the bugfixes are made, it's unlikely anyone would be caught out. (If you disable backcompat bits you probably care about bugfixes anyway!)<br></div><div><br></div><div>BTW, the git-svn cron job seems to be down again.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 9 Feb 2022 at 16:51, 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"><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" target="_blank">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>
_______________________________________________<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>