himselfv / wakan

Japanese and Chinese learning tool with dictionary
36 stars 7 forks source link

Settings lost when closing Wakan from "Language changed" #263

Open himselfv opened 10 years ago

himselfv commented 10 years ago

Original report by me.

Or a bunch of other such places which do Application.Terminate().

This is because most of the saving code is in fMenu.FormClose which doesn't get called. We can't just call it in FormDestroy because some things may already be torn down, and what about FlushUserData(), which may ask the user whether to save it, and let them cancel the operation?