<div dir="ltr"><div>Oh no!</div><div><br></div><div>I thought I'd screwed up badly when making that change. I see the problem was that saveenemydata was broken to begin with.<br></div><div><br></div><div>Well, if we're going to release dwimmercrafty+1, there are a few patches that could be backported... oh, too late.</div><div>It feels like too much effort to backport fixes to a release branch as we make them if we don't know whether there will be a bugfix release.<br></div><div><br></div><div>It would be good to have testcases which read all the data out of an enemy, hero, etc, to check that everything is in order. Then by running the latest nightly against old copies of autotest.rpg we can check that the data is updated correctly. But it's harder to check that all the upgrade routines we already have, for games older than 2017, work correctly. What we could do is craft an .rpg with an old version of custom, but import a modern .hs into it.<br></div><div>But we couldn't even have written testcases for this, because there are no script commands to access that data. "read enemy data" is intentionally limited to index 106 only. And writing individual script commands for all those settings is way too much work.<br></div><div>What we need are objects in HS so that we can access data with very minimal code required, and minimal documentation in plotdict.<br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 23 September 2017 at 09:29, James Paige <span dir="ltr"><<a href="mailto:Bob@hamsterrepublic.com" target="_blank">Bob@hamsterrepublic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I'll be releasing dwimmercrafty+1 with this fix. If I have time I'll do it today, otherwise it may have to wait for monday or tuesday<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 22, 2017 at 2:28 PM,  <span dir="ltr"><<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.<wbr>com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">james<br>
2017-09-22 14:28:14 -0700 (Fri, 22 Sep 2017)<br>
311<br>
Neither .bequest_attack nor .non_elem_counter_attack was being saved with the EnemyDef version of saveenemydata<br>
Ever since revision 9197, loadenemydata/saveenemy data is called on ever enemy record on every full upgrade<br>
so those two features were being deleted from all enemies when the rpg was opened in custom<br>
---<br>
U   wip/loading.rbas<br>
______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.o<wbr>rg</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org<wbr>/listinfo.cgi/ohrrpgce-motherh<wbr>amster.org</a><br>
</blockquote></div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org">ohrrpgce@lists.motherhamster.<wbr>org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.<wbr>org/listinfo.cgi/ohrrpgce-<wbr>motherhamster.org</a><br>
<br></blockquote></div><br></div>