Refactor save
Why (apart from the fix)?
Why (apart from the fix)?
Short answer is I don't really remember.
d6e5d18: I think this didn't fully fix the problem on it's own. The other changes were for fixing bugs caused by that change, XDG Base Directory Specification compliance, simplification, and readability.
af938fb: ensure_dev_dir and ensure_run_dir basically only called ensure_dir
09d9e49: mkdir_parent is more generic and, IMO, more useful.
6fee015: Honestly, this one might be mostly a style thing. I don't like goto. I think it should be avoided unless absolutely necessary. For me, this is more readable and more clearly indicates intent. You may disagree, and that's fine.
3b7b63b: converting from binary to text and from text back to binary is completely unnecessary.
412c991: Permissions.
8eea9f5: Permissions.
Let me know if you have other questions/concerns about any of the commits.
Thanks and welcome back.
Thanks, I'll take a look.