search

erkyrath / glulxe

The Glulx VM reference interpreter
http://eblong.com/zarf/glulx/
MIT License
113 stars 22 forks source link

Autorestore and ARC update for IosGlulxe (Request for comments, do not merge) #28

Closed angstsmurf closed 1 year ago

angstsmurf commented 1 year ago

To be used with https://github.com/erkyrath/iosglk/pull/5. This is an update to #27, bringing it up to date with the latest changes in IosGlk and IosGlulxe.

The changes are kind of invasive, so it might not make sense to merge it into the main repository. I am not sure what the best solution is here.

See also https://github.com/erkyrath/fizmo/pull/4

erkyrath commented 1 year ago

It is indeed messy. I don't want the core interpreter to have that much platform-specific code. Much less platform-specific code in a platform-specific language.

I suppose that leaves "fork it and merge core changes into the ObjC repository when needed".

angstsmurf commented 1 year ago

I'll just close this (and the fizmo PR). I think the submodules of https://github.com/angstsmurf/iosglulxe/tree/update and https://github.com/angstsmurf/iosfizmo/tree/update point where they should now, to the update branches of my forks.