eglaysher / rlvm

RealLive clone for Linux and OSX
http://rlvm.net
GNU General Public License v3.0
150 stars 25 forks source link

Some fixes for Little Busters russian patch #97

Closed xtsm closed 1 month ago

xtsm commented 1 month ago

I'm trying to resolve a couple of problems here.

eglaysher commented 1 month ago

The asan and the non-ascii window processing make sense, I would merge those immediately if they were a separate patch.

Still sort of confused about what's going on with rlbabel in this case; it's been 8 years since I've looked at this code at all.

xtsm commented 1 month ago

Ok, I've removed the two commits about line breaks. Might make another PR about it later, do you need any explanations from me?

I might also make more PRs as I keep encountering problems while playing the VN. Sorry for bringing this stuff up, I didn't even expect these patches to ever get merged because I thought this project was completely dead...

eglaysher commented 1 month ago

I mean, the project basically is dead, though I am still around and I'm fine merging any obvious build fixes or straight up bugfixes that are obviously correct.

eglaysher commented 1 month ago

In the case of the rlBabel code, my very vague recollection since it's been well over a decade, was that all of that code was not really that well thought out and the integration was just massive hacking things together. On one hand, the way you added a flag really feels bad and I'd bet breaks Japanese games, but otoh, it's not like the rest of the code in the text layout system was well thought out.

xtsm commented 1 month ago

Wdym by "breaks Japanese games"? I was trying to add this flag in such a way that it changes absolutely nothing unless consciously turned on. Did I miss something? When turned on, it definitely fixes my problem. The alternative solution would be to use standard msgothic font but it has square-shaped glyphs for cyrillic letters and reading that text makes me want to gouge my eyes out.

Otoh I didn't have a lot of time for thorough testing, maybe there would be more problems with this approach later in the game. The translation I'm currently using also looks pretty low-quality so maybe I'll just get an english one and call it a day...

Either way, thank you!

eglaysher commented 1 month ago

Sorry, when I said break Japanese games, my recollection is that it would blit one single extra Japanese fullwidth character past the bounding box of the text box.

xtsm commented 1 month ago

Maybe, idk. That's why it's a command line flag: unless the line breaks are bad, one shouldn't turn it on.

Of course ideally it should be solved by using the same line breaking code in rlbabel and text system, but that looks like too much work...