allanger/ardour
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Activity

remove duplicate code

Browse Source 
set_header_timeline_position() does exactly that. Also doing it
twice may have lead to a NULL-pointer dereference.
This commit is contained in:
Robin Gareus
2015-03-11 12:28:47 +01:00
parent 6d2189f9e3
commit ac23461ed9
1 changed files with 1 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
libs/ardour/sndfilesource.cc
View File

@@ -638,15 +638,6 @@ SndFileSource::setup_broadcast_info (framepos_t /*when*/, struct tm& now, time_t
set_header_timeline_position ();
if (!_broadcast_info->write_to_file (_sndfile)) {
error << string_compose (_("cannot set broadcast info for audio file %1 (%2); dropping broadcast info for this file"),
_path, _broadcast_info->get_error())
<< endmsg;
_flags = Flag (_flags & ~Broadcast);
delete _broadcast_info;
_broadcast_info = 0;
}
return 0;
}
@@ -656,6 +647,7 @@ SndFileSource::set_header_timeline_position ()
if (!(_flags & Broadcast)) {
return;
}
assert (_broadcast_info);
_broadcast_info->set_time_reference (_timeline_position);