I'll check out properties. Do they need to be on a TYPE? It can they exist in the global scope?<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"><div><div>(Changing herocount() is the first step of raising the reserve party limit)<br></div><div><br>"count" is clear, but I find "size" a bit ambiguous. If I change the name, I might rename some of the other new functions like active_party_size and is_active_party_slot too is I can figure out something consistent.<br></div><div><br></div>I frequently find when I search for something (often FB related), that if I've mentioned it on this mailinglist, then Google shows my own post near/at the top of the results. I don't know what their actual rank is; Google results are very personalised... well, it works! It's funny that gmail search is not so useful, I have to search the whole web to find the email I want :)<br><br></div>As an alternative to returning byref, you might be interested in properties too. They are syntactic sugar for getters/setters, like @property in python. Properties are very similar to byref-return functions, except that you 
can't pass any extra arguments to them, you need to write the getter and 
setter separately, and their setters can be much more flexible.<br>For example, I'd like to turn the x/y/z members of NPCInst into properties which return/set the position of its walkabout slice, so that you can move an NPC slice, without us having to update any other code. And obviously the same for heroes too, although I see now that will be more complicated. (Obviously, with both NPCs & heroes there are complications to work through, since the slice may not exist)<br><br><div class="gmail_extra"><br><div class="gmail_quote">On 1 June 2017 at 00:05, James Paige <span dir="ltr"><<a href="javascript:_e(%7B%7D,'cvml','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">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_part<wbr>y() 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 :)<div><div><br><br>On Wednesday, May 31, 2017, Ralph Versteegen <<a href="javascript:_e(%7B%7D,'cvml','teeemcee@gmail.com');" target="_blank">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>subversion@hamsterrepublic.co<wbr>m</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/ohrr<wbr>pgce/commits/743446cf368d9479a<wbr>4d8d216d0e3bbd6f9ae49fc?at=her<wbr>ocount</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></div></div>
<br>______________________________<wbr>_________________<br>
Ohrrpgce mailing list<br>
<a href="javascript:_e(%7B%7D,'cvml','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>
<br></blockquote></div><br></div></div>
</blockquote>