From ac23461ed9c066afce9baba711f5449671403933 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Wed, 11 Mar 2015 12:28:47 +0100 Subject: remove duplicate code set_header_timeline_position() does exactly that. Also doing it twice may have lead to a NULL-pointer dereference. --- libs/ardour/sndfilesource.cc | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) (limited to 'libs/ardour/sndfilesource.cc') diff --git a/libs/ardour/sndfilesource.cc b/libs/ardour/sndfilesource.cc index b93047761b..cc6cfdb4be 100644 --- a/libs/ardour/sndfilesource.cc +++ b/libs/ardour/sndfilesource.cc @@ -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); -- cgit v1.2.3