StatusBus keyed directly by status class

[rcahoon]

Description

As long as we don't do hot reloading or crazy things with ClassLoaders, there is exactly one Class object for each class. As such, it is safe to use Class<?> as HashMap keys because it will use the default Object implementations of equals and hashCode which are based on object identity. https://stackoverflow.com/a/54076411

Small performance gain over requiring hash and equality computations on strings.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Be detailed so that your code reviewer can understand exactly how much and what kinds of testing were done, and which might still be worthwhile to do.

• Unit tests: [Add your description here]
• Simulator testing: [Add your description here]
• On-robot bench testing: [Add your description here]
• On-robot field testing: [Add your description here]