search

L-Sherry / Localize-me

CCLoader mod to add locales
MIT License
5 stars 3 forks source link

fix bug in reserve_free_space which caused overwriting of the font #5

Closed dmitmel closed 4 years ago

dmitmel commented 4 years ago

Turns out you simply forgot to add width of the last character plus 2 - I copied that from the original implementation of reserve_free_space. Also, a name like this.allocated_space would be more appropriate instead of this.free_space from my understanding of the code.

L-Sherry commented 4 years ago

Well, since it should point to the first free space available (which is after all the allocated space), I find free_space to be appropriate. Anyway, good catch, thanks for finding out the problem.