diff options
| author | André Nusser <andre.nusser@googlemail.com> | 2016-01-29 16:45:31 +0100 | 
|---|---|---|
| committer | André Nusser <andre.nusser@googlemail.com> | 2016-01-29 16:45:31 +0100 | 
| commit | bbdde7f2d3c25e0fdc7c9e34c6dafcc8419dab3b (patch) | |
| tree | 5f3f186cb2b0cb20a7f4b23502b8223e4d242ecd /src | |
| parent | 7999bd2e087d5ee696899f1383c187b7e787069a (diff) | |
wip <blablabla>
Diffstat (limited to 'src')
| -rw-r--r-- | src/audiocachefile.cc | 6 | 
1 files changed, 3 insertions, 3 deletions
| diff --git a/src/audiocachefile.cc b/src/audiocachefile.cc index 3d566e7..0955b25 100644 --- a/src/audiocachefile.cc +++ b/src/audiocachefile.cc @@ -38,7 +38,7 @@ AudioCacheFile::AudioCacheFile(const std::string& filename,                                 std::vector<sample_t>& read_buffer)  	: filename(filename), read_buffer(read_buffer)  { -	std::memset(&sf_info, 0, sizeof(SF_INFO)); // XXX Is this really necessary? +	std::memset(&sf_info, 0, sizeof(SF_INFO));  	fh = sf_open(filename.c_str(), SFM_READ, &sf_info);  	if(!fh) @@ -50,7 +50,7 @@ AudioCacheFile::AudioCacheFile(const std::string& filename,  	if(sf_info.frames == 0)  	{ -		printf("sf_info.frames == 0\n"); // XXX Shouldn't that be debug output? +		WARN(cache, "sf_info.frames == 0\n");  	}  } @@ -135,7 +135,7 @@ AudioCacheFile& AudioCacheFiles::getFile(const std::string& filename)  	auto it = audiofiles.find(filename);  	if(it == audiofiles.end())  	{ -		cacheAudioFile = new AudioCacheFile(filename, read_buffer); // XXX why create a pointer and not move it into the map such that it is the owner? +		cacheAudioFile = new AudioCacheFile(filename, read_buffer);  		audiofiles.insert(std::make_pair(filename, cacheAudioFile));  	}  	else | 
