Open l3if3rik opened 1 month ago
Thank you very much for submitting! I am currently a bit busy but will try to review as soon as possible.
I have take a first look at the changes and posted some comments (I reviewed only a subset of changes so far, but already wanted to give feedback). I like the ideas very much, but I would really prefer to have separate submission for each of the concepts (at least for auto-creating directories, quick-save folders, post-processing actions), that would really simplify the review and allow to integrate some changes whilst still discussing others. The first two (auto-creating directories, quick-save folders) seem very useful and straightforward so I would like to integrate them quickly. The third idea (post-processing actions) also looks very good, but there are some conceptual and architectural difficulties (see details in comments I posted). Moreover, currently (at least for me), SVC just crashes when trying to start cutting.
Anyway, thanks again for taking the effort and posting your changes!
Thanks for taking the time to review all that! And sorry for the PR containing so much stuff. Initially, I was just tinkering around (it's been at least 10 years since I developed with .NET) and didn't even think of making a PR.
I will probably address the issues next weekend.
I would consider adding also a "cancel" option, that would mean the action is not constructed and the file is not being deleted.
Except for this, everything should be resolved now.
UnloadVideo()
can be used now with an unload/close button that would have to be added.
Hi @l3if3rik, just a short update from my side - I have seen you pushed lot of changes, but unfortunately I am very busy recently (plus vacation season) so had no time yet to review, and may still need some time to do it. Sorry for the delay and thanks for you effort!
This bug flew under the radar for quite some time. The files in videoFileArr
have their native mixed case names. The filename being searched is lowercase. I fixed that by doing a case-insensitive search like in GetNextPrevFileInDirectory()
.
A method like GetCurrentFileIndex()
would remove redundancy. It would not have a filename parameter, but use fileBeingPlayed
. It would fit in, if GetNextPrevFileInDirectory()
also dropped its argument currentFilePath
(it's not needed right now).
That's basically all the stuff I developed in several branches on my fork. While your program is the best for my purposes, it lacked some features that I really needed.
I do not know how to work with translations, so it is all hard-coded English, currently.
To clarify this further, here are some screenshots: