Janet Support

[iacore]

No description provided.

[joshgoebel]

Just to be clear this param is optional, not required.

[iacore]

that's not in the doc

[joshgoebel]

I would agree it's unclear.

btn([id]) -> pressed

The [] means the argument is optional. If no specific button is passed I believe the entire state of the button register (bitfield) is returned. (you can read the source to confirm)

I'm speaking largely of Lua, it might be useful to check and see if this has been handled correctly across all runtimes.

[joshgoebel]

Yes, btn and btnp have the same-ish behavior for 0 arguments:

https://github.com/nesbox/TIC-80/blob/main/src/api/lua.c#L601

[joshgoebel]

Docs updated.

[iacore]

btn() seem to always return true for me in the Lua version.

[AlecTroemel]

@locriacyber cool to see this MR, I actually spent some time on this same thing a while ago (branch here for comparison https://github.com/AlecTroemel/TIC-80/tree/charlotte_janet_support). I'd love to help get this MR ready however possible! some questions

• Do syntax errors contain line numbers & other info like relevant function names? I could never get that sorted
• You have an opinion on the tic80 function names that collide with the janet standard library functions? Specifically print and map?

[iacore]

Thanks for telling me. Seems like your work is further ahead.

• Janet prints error to stderr. It's hard coded in. Syntax error is considered compile-time error.
• runtime error is raised as signal
• I recommend making it a module, so the user can choose between (import tic80) or (use tic80)