You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Please provide proper function names (not "the next function as" - jut tell us which one you mean). Then describe them in greater detail.
Also always add which version of the game you are talking about.
(The following is for the patched US version)
I've checked sub_408B60 (which follows sub_408880) and it is indeed config writer. However, that is to be expected as it's probably in the same object file during the linking. As the writer is already documented it wasn't too important to also document the parser.
Documenting more config functions is currently not necessary - as the rough pattern has been described for other config files, so we can easily document them within a minute or so. We should care about the function which are harder to identify.
The idea behind documenting the config and menus was to document the addresses to which the config is loaded.
For example, we can easily tell that dword_EC86BC stores the model detail.
If a function uses strings, It's easy to spot what most functions do. We don't need issues or reminders for those.
The real problem is documenting the internals and functions which only do logic.
Feel free to still document it properly and send a pull request. It might be a good task for a starter to restructure it.
it's normally the next function as sub_408880().
The text was updated successfully, but these errors were encountered: