[Ohrrpgce] SVN: james/8376 Convert the mapping GOSUB block into mapeditor_mapping() SUB

[Ralph Versteegen]

Great! That one branch crossing warning in mapping has been annoying.
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.

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...

On 1 February 2017 at 07:47, <subversion at hamsterrepublic.com> wrote:

> james
> 2017-01-31 10:47:29 -0800 (Tue, 31 Jan 2017)
> 108
> Convert the mapping GOSUB block into mapeditor_mapping() SUB
> Also moved a bunch of stuff into MapEditState
> ---
> U   wip/custom_udts.bi
> U   wip/mapsubs.bas
> _______________________________________________
> Ohrrpgce mailing list
> ohrrpgce at lists.motherhamster.org
> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.motherhamster.org/pipermail/ohrrpgce-motherhamster.org/attachments/20170201/90f70388/attachment.htm>