pass by reference

Author: arahlin

core/include/core/G3Reader.h

@@ -9,10 +9,10 @@
 
 class G3Reader : public G3Module {
 public:
-	G3Reader(std::string filename, int n_frames_to_read = -1,
+	G3Reader(const std::string &filename, int n_frames_to_read = -1,
                  float timeout = -1., bool track_filename = false,
 	         size_t buffersize = 1024*1024);
-	G3Reader(std::vector<std::string> filenames, int n_frames_to_read = -1,
+	G3Reader(const std::vector<std::string> &filenames, int n_frames_to_read = -1,
                  float timeout = -1., bool track_filename = false,
 	         size_t buffersize = 1024*1024);
 	virtual ~G3Reader();
@@ -22,7 +22,7 @@ class G3Reader : public G3Module {
 	off_t Tell();
 
 private:
-	void StartFile(std::string path);
+	void StartFile(const std::string &path);
 	bool prefix_file_;
 	std::string cur_file_;
 	std::deque<std::string> filename_;

core/src/G3Reader.cxx

@@ -2,7 +2,7 @@
 #include <dataio.h>
 #include <G3Reader.h>
 
-G3Reader::G3Reader(std::string filename, int n_frames_to_read,
+G3Reader::G3Reader(const std::string &filename, int n_frames_to_read,
     float timeout, bool track_filename, size_t buffersize) :
     prefix_file_(false), stream_(nullptr), n_frames_to_read_(n_frames_to_read),
     n_frames_read_(0), n_frames_cur_(0), timeout_(timeout),
@@ -12,7 +12,7 @@ G3Reader::G3Reader(std::string filename, int n_frames_to_read,
 	StartFile(filename);
 }
 
-G3Reader::G3Reader(std::vector<std::string> filename, int n_frames_to_read,
+G3Reader::G3Reader(const std::vector<std::string> &filename, int n_frames_to_read,
     float timeout, bool track_filename, size_t buffersize) :
     prefix_file_(false), stream_(nullptr), n_frames_to_read_(n_frames_to_read),
     n_frames_read_(0), n_frames_cur_(0), timeout_(timeout),
@@ -34,7 +34,7 @@ G3Reader::~G3Reader()
 	g3_istream_close(stream_);
 }
 
-void G3Reader::StartFile(std::string path)
+void G3Reader::StartFile(const std::string &path)
 {
 	log_info("Starting file %s\n", path.c_str());
 	cur_file_ = path;