Fragile Code Cleanup

[zach2good]

I affirm:

• [x] that I agree to Project Topaz's Limited Contributor License Agreement, as written on this date
• [] that I've tested my code since the last commit in the PR, and will test after any later commits

NOT YET SANITY TESTED

I set up static analysis with Sonar, and started fixing all the things it complained about. Each commit explains what I'm doing.

The biggest thing is in the Packet System:

• session was poorly named, and shadowed a global session var -> renamed to PSession
• Every packet handler made copies of CBasicPacket, which isn't a big deal - CBasicPacket is just a bag of pointers, but we still shouldn't be doing that in the hot path 🤷‍♂️ Swapped it for move semantics gubbins

[TeoTwawki]

e is a pretty common variable convention to me, but maybe it's just my Java and Python experience has poisoned me. stuck_out_tongue

knee deep in the venom of experience, the young'uns need time to build up resistance