Reconnaisance.
These variables are unused.
Treat a bool like a bool, not an int.
Brace ALL the blocks!
Remove commented-out code.
Expand some abbreviations.
Split the sel_part variable into two variables.

Because one generic int variable should not have different meanings depending on where it is used!
Remove an unused local variable.
Correction to activity values size check.
Make sel_vehicle_part, ptank, and wheel into vehicle_part pointers instead of generic ints.
Convert sel_vpart_info from an int into a vpart_info pointer.

I will refactor this entire file even if it kills me.
Rename vpart_list (which will soon no longer be a list) to vehicle_part_types.
Remove the vpart_id enum and replace it with a map<string, vpart_info>.

This entire branch was created solely for this commit to be possible.

Also allows the ability to pass strings (without spaces in them) via player_activities.
Remove unused mfb definition.
Remove outdated comment, and use a flag over a hardcoded vpart id for lights.
Fix the narrowing warnings that were appearing on windows tiles builds
Merge remote branch 'origin/master' into veh-interact-pr

fixed Conflicts:
	pldata.h
	veh_interact.cpp
	veh_type.h
	veh_typedef.cpp
	vehicle.cpp
str vpart.id: Rewrite save/load code to new json save/load & legacy sstream loader, using autogenerated legacy_vpart_id vector where needed.
RFI: Take a perfectly good fusion emitter, and make a rifle out of it. Keep both arms.
Rename "first aid" item to "first aid kit".