theplu::utility::read_to_double (std::istream &is, std::vector< double > &vec)

[Jari Häkkinen]

Change the implementation of this function to follow C/C++ style treatment of files and streams. Also theplu::utility::read_to_int (std::istream &is, std::vector< int > &vec).

[Jari Häkkinen]

Or should we redesign this? Or even remove it?

[Markus Ringnér]

My vote goes for removing these. Peter?

[Peter]

I agree just because they are ugly. Are they used somewhere?

[Markus Ringnér]

They are not used anywhere within yat. Previously it has been said that Peter has external code using them.

Earlier they were used within yat by the vector/matrix istream-constructors, but when I updated these constructors to use stringstreams and to support various column-separator-characters they became obsolete. However, the functionality I implemented should in principle be separated from the constructors, but I think we should do this separation later and investigate if not the lexicographical parser parts of Boost should be utililized instead.

[Peter]

Well, that external code is not used anymore, so I still vote for removing the functions.

[Jari Häkkinen]

Ok, I'll remove them.

[Jari Häkkinen]

(In [723]) Fixes #10. Removed read_to*(std::vector..) functionality.