SnowyMouse / chimera

The update to Halo PC that never was
https://chimera.opencarnage.net
GNU General Public License v3.0
147 stars 27 forks source link

Check if the ini path exists #72

Closed MangoFizz closed 1 year ago

Aerocatia commented 3 years ago

Do we want to do this? Currently it works exactly like the -path argument in that the directory will be created if it does not exist.

MangoFizz commented 3 years ago

I found that if the parent folder doesn't exist, the game just crashes. Should I change it to check only the parent directory or just close this PR?

Edit: I will add a description next time, sorry! 😓

Aerocatia commented 3 years ago

We should handle that somehow. Does this issue affect the -path argument too?

MangoFizz commented 3 years ago

Trying that on a fresh install of the game, it seems to be handled by stock strings.dll

image

Aerocatia commented 3 years ago

It probably makes sense to handle it the same way.

MangoFizz commented 3 years ago

Great! I'll work on it tomorrow