Skip to content

Conversation

@jetrotal
Copy link
Contributor

@jetrotal jetrotal commented Oct 2, 2023

TPC commands for testing purposes:

@raw 99999, "@10110(""test"",[1 ,2, 3,4 ,5,6],0) //show dialog" 
@raw 99999, "@10310 ("""",[0 ,0, 100],0) //add 100 money" 
@raw 99999, "@10110(""@10110(\""test\"",[1 ,2, 3,4 ,5,6],0)"",[1 ,2, 3,4 ,5,6],0) //a collection of problematic syntax just to test how it deals with it"

Work in progress. I need a lot of help with this one.

It injects commands from 99999's com.string.

I can't parse proper syntax, since it isn't mapped anywhere.
But it can be useful alongside stringvars to load commands from txt files.

@jetrotal jetrotal force-pushed the Jetrotal-Scripting branch from 26dae40 to a31111b Compare October 2, 2023 06:26
TPC commands for testing purposes:

@raw 99999, "@10110(""test"",[1 ,2, 3,4 ,5,6],0) //show dialog"
@raw 99999, "@10310 ("""",[0 ,0, 100],0) //add 100 money"
@raw 99999, "@10110(""@10110(\""test\"",[1 ,2, 3,4 ,5,6],0)"",[1 ,2, 3,4 ,5,6],0) //show it's own syntax as dialog"

Work in progress. I need a lot of help with this one.

It injects commands from 99999's com.string.

I can't parse proper syntax, since it isn't mapped anywhere.
But it can be useful alongsize stringvars, to load commands from txt files.
@jetrotal jetrotal force-pushed the Jetrotal-Scripting branch from a31111b to 0802758 Compare October 2, 2023 17:35
@fdelapena
Copy link
Contributor

Looks promising, just some observations:

  • Because regex has portability issues, maybe is worth to replace it with some str functions.
  • Is 99999 used by an existing command? Otherwise maybe is better to use the maximum "reserved" value from easyrpg command IDs.

@jetrotal
Copy link
Contributor Author

jetrotal commented Nov 6, 2023

Not needed anymore, #3137 covers and improves this idea.

@jetrotal jetrotal closed this Nov 6, 2023
@jetrotal jetrotal deleted the Jetrotal-Scripting branch August 4, 2024 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants