diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2011-03-18 20:21:51 +0000 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2011-03-18 20:21:51 +0000 |
commit | ca84e02b48e0c36755963a35713d272c409d9317 (patch) | |
tree | 90f637b7a67431a75fe587fc8ddfee6d04b24de4 /libs/ardour/diskstream.cc | |
parent | 065b5ce8e7c0328153d087aa0209dbe6cb744d54 (diff) |
fix (?) capture alignment by making sure we use non-public latency information for playback latency, thus avoiding counting plugin latency twice
git-svn-id: svn://localhost/ardour2/branches/3.0@9168 d708f5d6-7413-0410-9779-e7cbd77b26cf
Diffstat (limited to 'libs/ardour/diskstream.cc')
-rw-r--r-- | libs/ardour/diskstream.cc | 37 |
1 files changed, 15 insertions, 22 deletions
diff --git a/libs/ardour/diskstream.cc b/libs/ardour/diskstream.cc index affda7b56a..bba28eeacd 100644 --- a/libs/ardour/diskstream.cc +++ b/libs/ardour/diskstream.cc @@ -655,35 +655,26 @@ Diskstream::check_record_status (framepos_t transport_frame, bool can_record) return; } - /* we transitioned to recording. lets see if its transport based or a punch */ - - first_recordable_frame = transport_frame + _capture_offset; - last_recordable_frame = max_framepos; capture_start_frame = _session.transport_frame(); + first_recordable_frame = capture_start_frame + _capture_offset; + last_recordable_frame = max_framepos; - /* in theory, we should be offsetting by _session.worst_playback_latency() when we adjust - for ExistingMaterial alignment. But that number includes the worst processor latency - across all routes, and each track will already be roll-delay adjusted to handle that. - so don't use worst_playback_latency(), just worst_output_latency() which covers - only downstream latency from IO ports. - */ - - DEBUG_TRACE (DEBUG::CaptureAlignment, string_compose ("%1: @ %7 basic FRF = %2 CSF = %4 CO = %5, EMO = %6 RD = %8\n", + DEBUG_TRACE (DEBUG::CaptureAlignment, string_compose ("%1: @ %7 (%9) FRF = %2 CSF = %4 CO = %5, EMO = %6 RD = %8 WOL %10 WTL %11\n", name(), first_recordable_frame, last_recordable_frame, capture_start_frame, _capture_offset, existing_material_offset, transport_frame, - _roll_delay)); + _roll_delay, + _session.transport_frame(), + _session.worst_output_latency(), + _session.worst_track_latency())); + if (_alignment_style == ExistingMaterial) { first_recordable_frame += existing_material_offset; DEBUG_TRACE (DEBUG::CaptureAlignment, string_compose ("\tshift FRF by EMO %1\n", first_recordable_frame)); - } else { - capture_start_frame += _roll_delay; - DEBUG_TRACE (DEBUG::CaptureAlignment, string_compose ("\tshift CFS by roll delay of %1 to %2\n", - _roll_delay, capture_start_frame)); - } + } prepare_record_status (capture_start_frame); @@ -703,13 +694,11 @@ Diskstream::check_record_status (framepos_t transport_frame, bool can_record) } else { /* punch out */ - last_recordable_frame = transport_frame + _capture_offset; + last_recordable_frame = _session.transport_frame() + _capture_offset; if (_alignment_style == ExistingMaterial) { last_recordable_frame += existing_material_offset; - } else { - last_recordable_frame += _roll_delay; - } + } } } } @@ -766,6 +755,10 @@ Diskstream::calculate_record_range(OverlapType ot, framepos_t transport_frame, f rec_offset = first_recordable_frame - transport_frame; break; } + + DEBUG_TRACE (DEBUG::CaptureAlignment, string_compose ("%1 rec? %2 @ %3 (for %4) FRF %5 LRF %6 : rf %7 @ %8\n", + _name, enum_2_string (ot), transport_frame, nframes, + first_recordable_frame, last_recordable_frame, rec_nframes, rec_offset)); } void |