fstream to basic_iostream in RinexStream

Question

jonyrock opened this issue 10 years ago · 4 comments

There is https://github.com/jonyrock/Gpstk/blob/master/dev/ext/lib/FileHandling/FFStream.hpp which base for all FF* classes. I suppose we need also change lib/FileHandling to lib/DataHandling

Answer 1 · 2015-02-02T11:37:45.000Z

@eugenyk I want to underline that we can't change FFStream interface, so the original constructors should remain.

Answer 2 · 2015-02-03T17:17:32.000Z

Answer 3 · 2015-02-04T09:53:49.000Z

@eugenyk please review the changes. Looks like it works. I think that the heart of my solution is
dev/ext/lib/FileHandling/FFStream.hpp (see all diff of this file) and here https://github.com/jonyrock/Gpstk/blob/bfd61eb6d30adb3bdfb07fc38bc8ff02bdcfeed0/dev/ext/lib/FileHandling/FFStream.cpp#L70

Answer 4 · 2015-03-22T11:46:36.000Z

Sources reviewed, comments added. std::iostream inheritance really looks more consistent