8bitkick / BBCMicroBot

Runs your toot on an 8-bit computer emulator
GNU General Public License v3.0
110 stars 8 forks source link

Don't load DFS by default #57

Open ojwb opened 1 year ago

ojwb commented 1 year ago

DFS was an add-on to the original hardware and isn't really useful with the bot. Loading it by default raises PAGE by 2816 bytes. Even in the best case of MODE7 and rocket mode, that's a 10% reduction in the available RAM to BBC BASIC. In MODE2 and with the GXR loaded it's a whopping 35% reduction.

Support enabling DFS via an emoji, which is probably only really useful as a way to allow easily running programs written for older versions of the bot which assume the value of PAGE or the screen contents.

Owlet ideally needs updating to match - happy to do that too.

8bitkick commented 1 year ago

Make sense. The only thing that concerns me is there are tweets with Owlet links out there that may break if we change the DFS default.

How about we give Owlet different default DFS behavior based on data URL version :

We make a v:1 the Twitter era (DFS is included)

We make a v:3 for Mastodon era (no DFS by default)

I started down this path with my URL shortener which is now live https://github.com/8bitkick/BBCMicroBot/blob/d0467a3040bfe64d55549c31488f6b234452c19a/mastodon.js#L74

ojwb commented 1 year ago

Good point, and the solution you suggest makes sense.

A new window really needs to default to v3 so that people writing new programs get something that matches the live bot.