nit: I'd prefer to see this in init list