These classes have virtual methods, but no virtual destructor, which causes warnings on some compilers.

Added the option to handle whether the user wants to iterate through the
assignment of button mappings or assign them one at a time.

fixed formatting issues and code style.

I excluded this option from the config file. This stopped the check box value and the boolean from becoming offset. Since the option should always start as false.

 This still causes an issue with the Wiimote input, since the class variable that keeps the state will be wiped, but the check box value will stay the same after closing/reopening without closing the entire Wiimote configuration. I am looking for a way to resolve this.

I also reduced wait time to 2.5 seconds vs. the 5  seconds previously. Seemed to be a little long.

These changes apparently did not go through.

This should fix the Wiimote issue.

Fixes an off by 1 issue related to double->float->double conversion, and eliminates numerous warnings.

InputCommon: Don't base default radius of analog sticks off of their name

Wrap dinput.h in a header defining DIRECTINPUT_VERSION instead of repeating it multiple places.

Also, bring WiiSaveCrypted closer to modern dolphin coding style.