Use the usual semantics for 'end' for Buffer::set_block, which in addition to actually making sense, allows GCC to optimize the loop.

git-svn-id: http://svn.drobilla.net/lad/trunk/ingen@4650 a436a847-0d15-0410-975c-d299462d15a1

1 files changed, 4 insertions, 5 deletions

diff --git a/src/server/Buffer.cpp b/src/server/Buffer.cpp
index 4de832e9..b10ccfc6 100644
--- a/src/server/Buffer.cpp
+++ b/src/server/Buffer.cpp
@@ -104,7 +104,7 @@ Buffer::clear()
 {
 	if (is_audio() || is_control()) {
 		_atom->size = _capacity - sizeof(LV2_Atom);
-		set_block(0, 0, nframes() - 1);
+		set_block(0, 0, nframes());
 	} else if (is_sequence()) {
 		_atom->size = sizeof(LV2_Atom_Sequence_Body);
 	}
@@ -125,13 +125,12 @@ Buffer::copy(Context& context, const Buffer* src)
 }
 
 void
-Buffer::set_block(Sample val, size_t start_offset, size_t end_offset)
+Buffer::set_block(Sample val, const SampleCount start, const SampleCount end)
 {
-	assert(end_offset >= start_offset);
-	assert(end_offset < nframes());
+	assert(end <= nframes());
 
 	Sample* const buf = samples();
-	for (size_t i = start_offset; i <= end_offset; ++i) {
+	for (SampleCount i = start; i < end; ++i) {
 		buf[i] = val;
 	}
 }