Working on the clean up. Several things are being caught so I’m fixing the stuff I missed. Some of the newer 64-bit functions are different so I’ve updated the variables for those as well. I’ve temporarily used unsigned long long but we’ll want to switch to std::uint64_t once we update the formatting strings, which will need to be done all at once as they’re different from the older form.
I don’t think those functions are used anyways but I’m updating them just in case. I’ll make a tracker note to check for 64-bit compatibility to reduce the likelihood of an out of range issue
Once it starts building again, I can merge it back to the main repository and we can go from there. 🙂
We’re more than likely going to need to run cppcheck multiple times as more stuff will be caught as we fix other things. For example, removing unused functions will cause other functions that were only being used in those to be marked as unused. It should catch most things in the initial pass, though.
