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.
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.
Merge pull request #1 from atomicdryad/veh-interact-pr

Merge to latest, rewrite save/load code to new json save/load & legacy sstream loader, using autogenerated legacy_vpart_id vector where needed.
Fix tiles to use the new map of vehicle part types.
Merge elimination of vpart_id enum, completing full jsonization of vparts. And player_activity.str_value
remote branch 'origin/pr/3173'