nit: these could be constants