From 30a9c2d05b422dc60e9f7c3e1cb35cd9d02accf2 Mon Sep 17 00:00:00 2001 From: Colin Fletcher Date: Mon, 1 Dec 2014 20:12:51 +0000 Subject: Comments in various call-sites of coverage() Comments in various call sites of Evoral::coverage() marking things I think are dubious (with XXX). Also straightened up the alignment of some ASCII art in libs/ardour/diskstream.cc --- libs/ardour/midi_diskstream.cc | 1 + 1 file changed, 1 insertion(+) (limited to 'libs/ardour/midi_diskstream.cc') diff --git a/libs/ardour/midi_diskstream.cc b/libs/ardour/midi_diskstream.cc index 9bc01b9bd4..27ec409b15 100644 --- a/libs/ardour/midi_diskstream.cc +++ b/libs/ardour/midi_diskstream.cc @@ -365,6 +365,7 @@ MidiDiskstream::process (BufferSet& bufs, framepos_t transport_frame, pframes_t if (nominally_recording || (re && was_recording && _session.get_record_enabled() && _session.config.get_punch_in())) { Evoral::OverlapType ot = Evoral::coverage (first_recordable_frame, last_recordable_frame, transport_frame, transport_frame + nframes); + // XXX should this be transport_frame + nframes - 1 ? coverage() expects its parameter ranges to include their end points calculate_record_range(ot, transport_frame, nframes, rec_nframes, rec_offset); /* For audio: not writing frames to the capture ringbuffer offsets -- cgit v1.2.3