minor style smell: use string, not String