Should these all be enums?