IMO it wouldn't make any big difference. The current solution has the benefit that the string values are just used in the parseScope method and therefore are highly mutable and extensible. Maybe we should think of relocating parseScope();