<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 3 February 2017 at 04:26, 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"><span class="">On Tue, Jan 31, 2017 at 4:19 PM, Ralph Versteegen <span dir="ltr"><<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><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><div>Great! That one branch crossing warning in mapping has been annoying.<br></div>I noticed that you didn't move certain arrays like lockedzonelist() into MapEditState. Don't forget that you can now put variable-length arrays in UDTs! You have to declare them like "lockedzonelist(any) as integer". The 'any' indicates that it's a varlen array with a single dimension; add more anys for more dimensions.<br></div></div></blockquote><div><br></div></span><div>Ooh! I did not know that. I have fixed it.<br><br></div><div>I also tried to move mode_tools_map() into MapEditState, but that caused a somewhat nonsensical error message in <a href="http://custom.udts.bi" target="_blank">custom.udts.bi</a> that did not seem to have anything to do with the part of the code that I was actually changing<br><br>/home/james/Dropbox/src/ohr/<wbr>wip/<a href="http://custom_udts.bi" target="_blank">custom_udts.bi</a>(344) error 14: Expected identifier, found 'MenuState' in 'zonemenustate as MenuState'<br>s<br><br></div></div></div></div></blockquote><div><br></div><div>That's saying that MenuState hasn't been declared. You need to put "#include "<a href="http://udts.bi">udts.bi</a>"" at the top of <a href="http://custom_udts.bi">custom_udts.bi</a>. A lot of our header files don't actually include the files they depend upon, but rley on them already being included.<br>If that doesn't work, it means there's a circular inclusion problem... but udts.bit certainly shouldn't include custom_udts. (BTW, fbc -showincludes is invaluable for debugging such circular includes)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><span class=""><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></div>Also, if you want to remove the GOSUBs in sprite_editor you can; I'm not currently working on it. Currently some of the tools in sprite_editor are implemented in GOSUBs, and some are just inlined in sprctrl. Since I converted it to Frames, most of the tools are just a single line plus boilerplate, so I was thinking of just inlining most rather than subifying, though consistency would be cleaner. Not sure if cleaning up sprctrl is even going to be beneficial long term...<br></div></blockquote><div><br></div></span><div>I'll tackle sprite_editor's gosubs if I have time.<br><br></div><div>Am I correct to guess that you *are* doing something with the map tile editor that I should avoid stepping on?<br></div></div></div></div></blockquote><div><br></div><div>No, I haven't touched the map tile editor.<br><br></div><div>I had assumed that eventually when sprite_editor gains the ability to edit sprites which use the master palette directly we could use it to edit map tiles as well, throwing away the separate map tile editor one.<br></div><div>But actually, the map tile editor seems to have cleaner code than the sprite editor, so it would be a pity to do so (although, as long as you're deleting code, you're reducing the amount of messy code!). I know you did a lot of cleanup some years ago. Maybe we can transplant in some of the good bits.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br>---<br></div><div>James <br></div><span class=""><div><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 class="m_-4880556318448619792gmail-HOEnZb"><div class="m_-4880556318448619792gmail-h5"><div class="gmail_extra"><div class="gmail_quote">On 1 February 2017 at 07:47, <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>
2017-01-31 10:47:29 -0800 (Tue, 31 Jan 2017)<br>
108<br>
Convert the mapping GOSUB block into mapeditor_mapping() SUB<br>
Also moved a bunch of stuff into MapEditState<br>
---<br>
U wip/<a href="http://custom_udts.bi" rel="noreferrer" target="_blank">custom_udts.bi</a><br>
U wip/mapsubs.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></span></div><br></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></div>