-
Notifications
You must be signed in to change notification settings - Fork 11
Improvements and fixes #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
* From updates: Added pause, fixed pause, cleaned up code and presentation
* Gave more specific names to ambiguous variables * Added some comments * Used string interpolation instead of concatenation
* Allows the use of RTE with Zeku, G, and any other large-file characters (<4000kB). * Previous limit was 2000kB. Probably don't need more than 3000kB, but this gives some wiggle room.
|
Note about some of the commits: There was a weird end-of-line inconsistency (CLRF) thing that caused some of the diffs to show entire files being replaced, even if it was just a couple lines. 😓 |
…ad in old names * Successful test of name conversion!
|
Thanks @DoctorBearPhD! All of these changes looks good to me. The code was quite messy in places, I see you even cleaned it up a bit. "* Successful test of name conversion!" means that there are no breaking changes since last version, right? If so, that's excellent! About the diffs, I can't actually see what you changed in the files that are "replaced". It would be better if you could fix that somehow. It's going to be hard to find out what needs to be looked at later if your changes introduces new bugs. Does "Single merged .exe file for each program (no separate .dll's)" mean that there's no separate MoveLib.dll anymore? I don't know if others are using movelib in their projects. I think sfvsim (https://gitlab.com/loic.petit/sfvsim) used to use it, but Loïc made his own tool I think (https://gitlab.com/loic.petit/sfvfile)? Would this change make it harder for others to use movelib in their projects? Over to something completely different: I think I've seen your nick before. Are you by any chance working on Mysterious Mod? I think MM is great and also one of the biggest projects that use MoveTool. Handing the MoveTool project over to someone working on MM feels like a natural move to me. If either of you, DoctorBearPhD and Loïc, or both, want to take over for me, that would be great. In that case, please clone this repository somewhere and give me the link. I will put it at the top of the readme so that everyone who comes here knows MoveTool moved and to where. |
|
Yeah I made my own movetool for sfvsim but I'm still using a modified version of movetool for sfvdiff (using bson not json) and for exploratory digging. I think that most of the work on Movetool has been made lately by @DoctorBearPhD (I only made a small change when season 3 started). Maybe we could just use his fork and start from there ? I'm not a MM contributor but he is and I think that MM is the prime user of Movetool now. That said, @DoctorBearPhD, you know that if you need help you know where to find me :) |
|
That sounds good to me, @WydD. |
|
Correct; no breaking changes since last version. Sure, I'd love to take over as maintainer of MoveTool. 👍 |
|
In the diff viewer, it looks like you can click |
|
Thanks, that's good to hear! A GUI would be nice for sure, I know lots of people get scared by the json. Getting the dll from this repository means people would get an outdated version after a while though. Anyone can just get the updated source and build it of course, so I guess it's fine. You're right about the Diff Settings. I didn't know about that. I wouldn't worry about the "replaced" files then. As I've said, I'm not really into SF5, but I do watch some MM-stuff once in a while. It looks great. It looks like more fun than the regular game to be honest. Keep up the great work guys! |
I made some changes and fixes. Take a look! 🤓
Improvements include: