Streaming: Use const reference in place of copying

* Thanks to Markus for reporting.
This commit is contained in:
Dario Casalinuovo
2016-06-25 01:13:10 +02:00
parent 7f28cdba0c
commit 4dd09daff7
9 changed files with 18 additions and 16 deletions
+7 -5
View File
@@ -72,8 +72,8 @@ public:
// Additional constructors used to stream data from protocols // Additional constructors used to stream data from protocols
// supported by the Streamer API // supported by the Streamer API
BMediaFile(BUrl url); BMediaFile(const BUrl& url);
BMediaFile(BUrl url, int32 flags); BMediaFile(const BUrl& url, int32 flags);
// Read-Write streaming constructor // Read-Write streaming constructor
BMediaFile(BUrl destination, BMediaFile(BUrl destination,
const media_file_format* mfi, const media_file_format* mfi,
@@ -84,7 +84,7 @@ public:
status_t SetTo(const entry_ref* ref); status_t SetTo(const entry_ref* ref);
status_t SetTo(BDataIO* destination); status_t SetTo(BDataIO* destination);
// The streaming equivalent of SetTo // The streaming equivalent of SetTo
status_t SetTo(BUrl url); status_t SetTo(const BUrl& url);
status_t InitCheck() const; status_t InitCheck() const;
@@ -179,9 +179,11 @@ private:
void _Init(); void _Init();
void _UnInit(); void _UnInit();
void _InitReader(BDataIO* source, BUrl* url = NULL, void _InitReader(BDataIO* source,
const BUrl* url = NULL,
int32 flags = 0); int32 flags = 0);
void _InitWriter(BDataIO* target, BUrl* url, void _InitWriter(BDataIO* target,
const BUrl* url,
const media_file_format* fileFormat, const media_file_format* fileFormat,
int32 flags); int32 flags);
+1 -1
View File
@@ -40,7 +40,7 @@ struct stream_info {
class MediaExtractor { class MediaExtractor {
public: public:
MediaExtractor(BDataIO* source, int32 flags); MediaExtractor(BDataIO* source, int32 flags);
MediaExtractor(BUrl url, int32 flags); MediaExtractor(const BUrl& url, int32 flags);
~MediaExtractor(); ~MediaExtractor();
+1 -1
View File
@@ -18,7 +18,7 @@ public:
Streamer(); Streamer();
virtual ~Streamer(); virtual ~Streamer();
virtual status_t Sniff(BUrl url, BDataIO** source) = 0; virtual status_t Sniff(const BUrl& url, BDataIO** source) = 0;
private: private:
virtual void _ReservedStreamer1(); virtual void _ReservedStreamer1();
virtual void _ReservedStreamer2(); virtual void _ReservedStreamer2();
@@ -20,7 +20,7 @@ HTTPStreamer::~HTTPStreamer()
status_t status_t
HTTPStreamer::Sniff(BUrl url, BDataIO** source) HTTPStreamer::Sniff(const BUrl& url, BDataIO** source)
{ {
HTTPMediaIO* ret = new HTTPMediaIO(url); HTTPMediaIO* ret = new HTTPMediaIO(url);
if (ret->InitCheck() == B_OK) { if (ret->InitCheck() == B_OK) {
@@ -14,7 +14,7 @@ public:
HTTPStreamer(); HTTPStreamer();
virtual ~HTTPStreamer(); virtual ~HTTPStreamer();
virtual status_t Sniff(BUrl url, BDataIO** source); virtual status_t Sniff(const BUrl& url, BDataIO** source);
}; };
@@ -15,7 +15,7 @@ RTSPStreamer::~RTSPStreamer()
status_t status_t
RTSPStreamer::Sniff(BUrl url, BDataIO** source) RTSPStreamer::Sniff(const BUrl& url, BDataIO** source)
{ {
RTSPMediaIO* ret = new RTSPMediaIO(url); RTSPMediaIO* ret = new RTSPMediaIO(url);
if (ret->InitCheck() == B_OK) { if (ret->InitCheck() == B_OK) {
@@ -9,7 +9,7 @@ public:
RTSPStreamer(); RTSPStreamer();
virtual ~RTSPStreamer(); virtual ~RTSPStreamer();
virtual status_t Sniff(BUrl url, BDataIO** source); virtual status_t Sniff(const BUrl& url, BDataIO** source);
}; };
+1 -1
View File
@@ -64,7 +64,7 @@ MediaExtractor::MediaExtractor(BDataIO* source, int32 flags)
} }
MediaExtractor::MediaExtractor(BUrl url, int32 flags) MediaExtractor::MediaExtractor(const BUrl& url, int32 flags)
: :
fExtractorThread(-1), fExtractorThread(-1),
fReader(NULL), fReader(NULL),
+4 -4
View File
@@ -82,7 +82,7 @@ BMediaFile::BMediaFile(const media_file_format* mfi, int32 flags)
} }
BMediaFile::BMediaFile(BUrl url) BMediaFile::BMediaFile(const BUrl& url)
{ {
CALLED(); CALLED();
fDeleteSource = true; fDeleteSource = true;
@@ -91,7 +91,7 @@ BMediaFile::BMediaFile(BUrl url)
} }
BMediaFile::BMediaFile(BUrl url, int32 flags) BMediaFile::BMediaFile(const BUrl& url, int32 flags)
{ {
CALLED(); CALLED();
fDeleteSource = true; fDeleteSource = true;
@@ -471,7 +471,7 @@ BMediaFile::_UnInit()
void void
BMediaFile::_InitReader(BDataIO* source, BUrl* url, int32 flags) BMediaFile::_InitReader(BDataIO* source, const BUrl* url, int32 flags)
{ {
CALLED(); CALLED();
@@ -512,7 +512,7 @@ BMediaFile::_InitReader(BDataIO* source, BUrl* url, int32 flags)
void void
BMediaFile::_InitWriter(BDataIO* target, BUrl* url, BMediaFile::_InitWriter(BDataIO* target, const BUrl* url,
const media_file_format* fileFormat, int32 flags) const media_file_format* fileFormat, int32 flags)
{ {
CALLED(); CALLED();