Add comment about parameters to coverage() including the end point.
authorColin Fletcher <colin.m.fletcher@googlemail.com>
Mon, 1 Dec 2014 20:06:08 +0000 (20:06 +0000)
committerColin Fletcher <colin.m.fletcher@googlemail.com>
Mon, 1 Dec 2014 20:06:08 +0000 (20:06 +0000)
gtk2_ardour/editor_ops.cc

index f0de126c6735046a66bfa37be53d424fe17dd106..e8ca0e1ee2f7754256fd5f579340cd402c62ed27 100644 (file)
@@ -2703,6 +2703,7 @@ static void
 add_if_covered (RegionView* rv, const AudioRange* ar, RegionSelection* rs)
 {
        switch (rv->region()->coverage (ar->start, ar->end - 1)) {
+       // n.b. -1 because AudioRange::end is one past the end, but coverage expects inclusive ranges
        case Evoral::OverlapNone:
                break;
        default: