is this correct?  I think it includes the last whitespace/eol character,