Rewrite the input buffer. #862
Replies: 5 comments
-
I closed the issue about keeping 0.94-0.97 input parser as an optional. Once Gacel releases his own input parser it would be good to have an option on the config.json to choose between the two, while Gacel one will be the one turned on by default. |
Beta Was this translation helpful? Give feedback.
-
I have a similar problem as in #116 with my chars EDIT |
Beta Was this translation helpful? Give feedback.
-
For what it's worth, here's a 1:1 port of Turtlewax that I wrote in Go. I'm not sure how one could integrate this into our current input system but it could be used for inspiration by anyone who'd like to pick this project up. |
Beta Was this translation helpful? Give feedback.
-
I think the problem with referencing a custom buffer is that they approach the problem in a different way. They're coded at the character level in a "if cmd file command is performed, set variable" logic. What the Ikemen parser needs to do (well, already does) is compare the player inputs to the list of commands that are defined in the cmd file. After Plasmoid Thunder pointed out #1280 to me, I think the current parser will need at least a rewrite of the "ReadCommand" function. The problem of reading inputs was approached in the wrong way from the getgo so some parts of the code aren't salvageable if one wants to make a parser like Mugen's. I'd be happy to help with this but my knowledge of Golang is very basic and this needs to be tackled by someone with a certain level of mastery over it. I'd still like to help with testing and brainstorming though. |
Beta Was this translation helpful? Give feedback.
-
So I've been working on the input code lately. So far I was able to fix enough bugs to be somewhat confident in saying that it won't need to be rewritten. The bugs that it has are fixable (most are fixed already), and the main issue it had in terms of expandability was being a confusing code. Suehiro understandably wrote the code for himself so it has very little annotations and such. But it's a great code. We just have to finish what he started. |
Beta Was this translation helpful? Give feedback.
-
Currently the input buffer present some problems that make it harder to execute commands.
This is show on these cases like #116.
And any improvements like ones done by NeatUnsou break the functionality of current one.
So to solve problems and allow improvements in the future a full command buffer rewrite is needed.
As compatibly with MUGEN is pority this new buffer would be fully backwards compatible with MUGEN chars.
Beta Was this translation helpful? Give feedback.
All reactions