X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Ftime_selection.cc;h=2f40dcb8e4afe0a06c80f963193fa94660bb1caf;hb=f66f7ba136c8588e47b3381ab07353e85f4100bf;hp=c0a05abc458ff13fa2edb0fad6425f5b8f0cfb86;hpb=fedf3d34f32264ac57c6a222b678dc90f2bb1a88;p=ardour.git

diff --git a/gtk2_ardour/time_selection.cc b/gtk2_ardour/time_selection.cc
index c0a05abc45..2f40dcb8e4 100644
--- a/gtk2_ardour/time_selection.cc
+++ b/gtk2_ardour/time_selection.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2003-2004 Paul Davis 
+    Copyright (C) 2003-2004 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -15,17 +15,16 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <algorithm>
 
-#include <pbd/error.h>
-#include <ardour/ardour.h>
+#include "pbd/error.h"
+#include "ardour/types.h"
 
 #include "time_selection.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace ARDOUR;
 using namespace PBD;
@@ -39,7 +38,7 @@ TimeSelection::operator[] (uint32_t which)
 		}
 	}
 	fatal << string_compose (_("programming error: request for non-existent audio range (%1)!"), which) << endmsg;
-	/*NOTREACHED*/
+	abort(); /*NOTREACHED*/
 	return *(new AudioRange(0,0,0)); /* keep the compiler happy; never called */
 }
 
@@ -56,9 +55,9 @@ TimeSelection::consolidate ()
 				continue;
 			}
 
-			if ((*a).coverage ((*b).start, (*b).end) != OverlapNone) {
-				(*a).start = std::min ((*a).start, (*b).start);
-				(*a).end = std::max ((*a).end, (*b).end);
+			if (a->coverage (b->start, b->end) != Evoral::OverlapNone) {
+				a->start = std::min (a->start, b->start);
+				a->end = std::max (a->end, b->end);
 				erase (b);
 				changed = true;
 				goto restart;
@@ -67,16 +66,16 @@ TimeSelection::consolidate ()
 	}
 
 	return changed;
-}		
+}
 
-nframes_t 
+framepos_t
 TimeSelection::start ()
 {
 	if (empty()) {
 		return 0;
 	}
 
-	nframes_t first = max_frames;
+	framepos_t first = max_framepos;
 
 	for (std::list<AudioRange>::iterator i = begin(); i != end(); ++i) {
 		if ((*i).start < first) {
@@ -86,10 +85,10 @@ TimeSelection::start ()
 	return first;
 }
 
-nframes_t 
+framepos_t
 TimeSelection::end_frame ()
 {
-	nframes_t last = 0;
+	framepos_t last = 0;
 
 	/* XXX make this work like RegionSelection: no linear search needed */
 
@@ -101,9 +100,12 @@ TimeSelection::end_frame ()
 	return last;
 }
 
-nframes_t
+framecnt_t
 TimeSelection::length()
 {
+	if (empty()) {
+		return 0;
+	}
+
 	return end_frame() - start() + 1;
 }
-