Minor fixes

[cmdoug]

No description provided.

[cmdoug]

I am trying to understand things well enough to fix #319, but I am pretty perplexed and making little progress. These are just some mundane things I noticed while poking around.

[cmdoug]

I'm going to keep this marked as a draft to have a catchall branch for minor changes as I poke around further

[cmdoug]

Hi @prj-, do you have thoughts on whether we merge these minor changes to purge all of this "dead logic"? Or is it really being held there for some reason? For example, see my review comment above -- it looks like maybe some of the logic was meant as a switch for testing, and it is unclear whether it is supposed to be active or unactive in the actual code.
Also, on a side note, is there any plan to bring in some form of enforced format (e.g. Clang format) for the FF codebase? Something like this would be a helpful add I think.

[prj-]

I'm going to keep this marked as a draft

Is it still a draft or is it now ready for review?

[cmdoug]

Can be reviewed. Its pretty minor stuff mostly, but some of the changes make me wonder about the original intent.

[prj-]

About the use of clang-format, it should be used. Unfortunately, I'm not sure what happened last time, but it messed up some of the repo, see f715746.