Opened 15 years ago

Closed 14 years ago

#10 closed defect (fixed)

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

Reported by: Jari Häkkinen Owned by: Jari Häkkinen
Priority: major Milestone: yat 0.2
Component: utility Version: trunk
Keywords: Cc:

Description (last modified by 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).

Change History (8)

comment:1 Changed 15 years ago by Jari Häkkinen

Description: modified (diff)

comment:2 Changed 14 years ago by Jari Häkkinen

Milestone: later0.2

Or should we redesign this? Or even remove it?

comment:3 Changed 14 years ago by Markus Ringnér

My vote goes for removing these. Peter?

comment:4 Changed 14 years ago by Peter

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

comment:5 Changed 14 years ago by 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.

comment:6 Changed 14 years ago by Peter

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

comment:7 Changed 14 years ago by Jari Häkkinen

Status: newassigned

Ok, I'll remove them.

comment:8 Changed 14 years ago by Jari Häkkinen

Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.