Should we be defining this in a header?