AZO234 / NP2kai

Neko Project II kai
http://domisan.sakura.ne.jp/article/np2kai/np2kai.html
MIT License
246 stars 61 forks source link

[SDL,X] Fixed the issue of not being able to set more than 256MB of memory. #125

Closed miyamoto999 closed 3 years ago

miyamoto999 commented 3 years ago

今回もたまたま見つけたのですが😅

SDL、Xで設定ファイルを読み書きするところで、メモリサイズをUINT8として扱っていたので256MB以上設定できなかったのでUINT16で扱うように改修しました。あと、SDLではメニューにチェックを付けるところでもUINT8として扱っていたのでそれも改修しました。

よくわからなかったのですが、INIMAX_*はどういう意味があるんでしょうか? INIMAX_UINT16がなくて、ソースを追ってみてもよくわからなかったのでINITYPE_UINT16にとりあえず変更しています。


I just happened to find it again this time 😅.

Because the memory size was treated as UINT8, it was not possible to set more than 256MB, so it was fixed to treat with UINT16. Also, in the SDL version, even the part to check the menu is treated as UINT8, so that is also fixed.

I wasn't sure, but what does INIMAX_* mean?

I couldn't find INIMAX_UINT16 and I didn't understand the source, For the time being, I changed it to INITYPE_UINT16.

AZO234 commented 3 years ago

Wow... It's bug... 😫 Thank you!