I'm in the process of deciding which races have pupils and which ones don't.
The humans, of course, need to have pupils. goblins and elves, not so much. Not sure about half-elves though.
Olin and Urist are both mountain dwarves, and they have a special third eyelid that keeps out dust, so for the most part their pupils are not visible. Hill dwarves (dwarrow) lack this feature, as they spend less time below ground. they have large and shiny eyes with little white visible.