That doesn't look like it will conflict with what I am working on.<div><br></div><div>I probably would have named active_hero_count() count_of_heroes_in_active_party() which is silly long :)</div><div><br></div><div>Hopefully I'll have the patch that replaces most direct access of cats() with wrapper functions ready later today</div><div><br></div><div>By the way, I love functions that return byref!</div><div><br></div><div>I went googling for the correct syntax for disambiguating assignment to a byref return value from comparison, and the test-cases you wrote for byref functions was ranked higher than the actual docs :)<br><br>On Wednesday, May 31, 2017, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com">teeemcee@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 31 May 2017 at 09:20,  <span dir="ltr"><<a href="javascript:_e(%7B%7D,'cvml','subversion@hamsterrepublic.com');" target="_blank">subversion@hamsterrepublic.<wbr>com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">james<br>
2017-05-30 14:20:42 -0700 (Tue, 30 May 2017)<br>
161<br>
Begin cleanup of hero position. Remove catx() caty() catz() catd() arrays and replace them with cats() array of type CaterpillarHistory<br>
(More cleanup to follow)<br>
---<br>
U   wip/game.bas<br>
U   wip/<a href="http://game_udts.bi" rel="noreferrer" target="_blank">game_udts.bi</a><br>
U   wip/<a href="http://gglobals.bi" rel="noreferrer" target="_blank">gglobals.bi</a><br>
U   wip/moresubs.rbas<br>
U   wip/savegame.rbas<br>
U   wip/scriptcommands.bas<br>
U   wip/walkabouts.bas<br>
U   wip/<a href="http://walkabouts.bi" rel="noreferrer" target="_blank">walkabouts.bi</a><br>
U   wip/yetmore2.bas<br></blockquote></div><br></div><div class="gmail_extra">I was going to say that any "fix" to the caterpillar hero speed problem would be largely a rewrite, but you really are doing it!</div><div class="gmail_extra"><br>I was concerned that this might clash with my herocount() work, but so far it's fine, and probably the rest of the change won't conflict either. I'll try to commit that soon (turns out there are various unrelated conflicts).<br><br><a href="https://bitbucket.org/rbv/ohrrpgce/commits/743446cf368d9479a4d8d216d0e3bbd6f9ae49fc?at=herocount" target="_blank">https://bitbucket.org/rbv/<wbr>ohrrpgce/commits/<wbr>743446cf368d9479a4d8d216d0e3bb<wbr>d6f9ae49fc?at=herocount</a><br><br></div><div class="gmail_extra">(The main reason I didn't finish was that I didn't like the name active_hero_count...)<br></div></div>
</blockquote></div>