X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fmidi_streamview.cc;h=f3b418fc99fede4afecd42f3b9a967ccce40ce63;hb=7b155267505af3fedffc7972b08cb784c97b7e74;hp=723b52220210e0b3e28e972af24e6a5632135a94;hpb=d08e3b94c5ef75ce5d4d28f3636faa8af4661e17;p=ardour.git

diff --git a/gtk2_ardour/midi_streamview.cc b/gtk2_ardour/midi_streamview.cc
index 723b522202..f3b418fc99 100644
--- a/gtk2_ardour/midi_streamview.cc
+++ b/gtk2_ardour/midi_streamview.cc
@@ -452,6 +452,20 @@ MidiStreamView::setup_rec_box ()
 				
 				plist.add (ARDOUR::Properties::start, start);
 				plist.add (ARDOUR::Properties::length, 1);
+				/* Just above we're setting this nascent region's length to 1.  I think this
+				   is so that the RegionView gets created with a non-zero width, as apparently
+				   creating a RegionView with a zero width causes it never to be displayed
+				   (there is a warning in TimeAxisViewItem::init about this).  However, we
+				   must also set length_beats to something non-zero, otherwise the frame length
+				   of 1 causes length_beats to be set to some small quantity << 1.  Then
+				   when the position is set up below, this length_beats is used to recompute
+				   length using Tempo::framepos_plus_bbt, which is slightly innacurate for small
+				   beats values (see comment in Tempo::framepos_plus_bbt) [mostly because
+				   it converts floating point beats to bars, beats and integer ticks].  The upshot
+				   of which being that length gets set back to 0, meaning no region view is ever
+				   seen, meaning no MIDI notes during record (#3820).
+				*/
+				plist.add (ARDOUR::Properties::length_beats, 1);
 				plist.add (ARDOUR::Properties::name, string());
 				plist.add (ARDOUR::Properties::layer, 0);