Replace split_whitespace().collect() calls

[nils-ohlmeier]

Instead of using split_whitespace().collect() to get a temporary vector just use split_whitespace() and then the iterator with .next() for parsing.

[na-g]

The code currently checks the length of the result of the collect(). AFAIK there is no way of doing that with an Iter without consuming the Iter.

[nils-ohlmeier]

Correct.
But in hindsight I think these length checks are somewhat stupid. They make testing a lot harder then it needs to be. For some of the attributed the amount of tokens depends on one of the tokens. And when you consume the tokens from an Iter the parser can provide a precise error message what is wrong with line instead of just "it's too short".
So basically I think we should not check the length at all and instead consume from the .next() calls.