<div dir="ltr"><div><div>(Main reply at "Re: SVN: james/8209 SUBify the GOSUB/RETRACE block that loaded the hero formation previews...")<br><br></div>But SliceTable is not used in Custom anyway? Well, on checking, it turns out that it is used in exactly two places: LookupSlice is the only function in slices.bas (aside from SetupGameSlices and DestroyGameSlices) that accesses SliceTable, and also enemydata in subs.rbas does "SetSliceParent(preview, SliceTable.Root)", which appears to be an error, since SliceTable.Root is NULL.<br><br></div>I've always been unhappy about SliceTable/SetupGameSlices/DestroyGameSlices existing in slices.bas instead of Game-specific source files; if possible to get rid of the LookupSlice dependency I would move it into GameState.<br><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On 12 November 2016 at 11:04, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I added this because Taoki was getting a crash<br><br>$1 = 0x8801c0 <__fb_errmsg> "\nAborting due to runtime error 7 (null pointer access) at line 796 of slices.bas::LOOKUPSLICE()\n\n"<br><pre>Whenever he would highlight an "Empty" slot in the "Construct Formation Sets" menu on Linux 64 bit<br><br></pre><pre>(I was unable to reproduce this on 32 bit linux)<br><br>---<br></pre><pre>James<br><br></pre><pre><br></pre><pre><br></pre><br></div><div class="gmail-m_-1155948914289812377HOEnZb"><div class="gmail-m_-1155948914289812377h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 11, 2016 at 2:00 PM,  <span dir="ltr"><<a href="mailto:subversion@hamsterrepublic.com" target="_blank">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>
2016-11-11 14:00:46 -0800 (Fri, 11 Nov 2016)<br>
112<br>
Add an extra null pointer check in LookupSlice, in case the default start slice of SliceTable.root is also null<br>
---<br>
U   wip/slices.bas<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" 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></div></div></div>