X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fevoral%2Fevoral%2FTimeConverter.hpp;h=6c02507c174aaf2b52544104b3be038e9dca3d96;hb=22fd92f97c5ff2bb1890a3b88085bed5c035498e;hp=25371a18bf88700a64ba477ccfdea78b4bc03ec1;hpb=5e3ca4db5cc356385e520643cfd279f393db4b51;p=ardour.git

diff --git a/libs/evoral/evoral/TimeConverter.hpp b/libs/evoral/evoral/TimeConverter.hpp
index 25371a18bf..6c02507c17 100644
--- a/libs/evoral/evoral/TimeConverter.hpp
+++ b/libs/evoral/evoral/TimeConverter.hpp
@@ -1,5 +1,5 @@
 /* This file is part of Evoral.
- * Copyright (C) 2009 Dave Robillard <http://drobilla.net>
+ * Copyright (C) 2009 David Robillard <http://drobilla.net>
  * Copyright (C) 2009 Paul Davis
  *
  * Evoral is free software; you can redistribute it and/or modify it under the
@@ -19,18 +19,27 @@
 #ifndef EVORAL_TIME_CONVERTER_HPP
 #define EVORAL_TIME_CONVERTER_HPP
 
+#include "evoral/visibility.h"
+
 namespace Evoral {
 
 /** A bidirectional converter between two different time units.
  *
  * Think of the conversion method names as if they are written in-between
  * the two template parameters (i.e. "A <name> B").
+ *
+ * _origin_b should be the origin for conversion in the units of B.
+ * That is, there is some point in time _origin_b, such that:
+ *
+ *    to()   converts a time _origin_b + a into an offset from _origin_b in units of B.
+ *    from() converts a time _origin_b + b into an offset from _origin_b in units of A.
  */
 template<typename A, typename B>
-class TimeConverter {
+class LIBEVORAL_TEMPLATE_API TimeConverter {
 public:
-	TimeConverter (B ob = 0) : _origin_b (ob) {}
-	virtual ~TimeConverter() {}
+	TimeConverter () : _origin_b (0) {}
+	TimeConverter (B ob) : _origin_b (ob) {}
+	virtual ~TimeConverter();
 
 	/** Convert A time to B time (A to B) */
 	virtual B to(A a) const = 0;
@@ -41,7 +50,7 @@ public:
 	B origin_b () const {
 		return _origin_b;
 	}
-	
+
 	void set_origin_b (B o) {
 		_origin_b = o;
 	}
@@ -51,11 +60,17 @@ protected:
 };
 
 
-/** A stub TimeConverter that simple statically casts between types. */
+/** A stub TimeConverter that simple statically casts between types.
+ *  _origin_b has no bearing here, as there is no time conversion
+ *  going on.
+ */
 template<typename A, typename B>
-class IdentityConverter : public TimeConverter<A,B> {
-	B to(A a)   const { return static_cast<B>(a); }
-	A from(B b) const { return static_cast<A>(b); }
+class LIBEVORAL_TEMPLATE_API IdentityConverter : public TimeConverter<A,B> {
+  public:
+	IdentityConverter() {}
+
+	B to(A a)   const;
+	A from(B b) const;
 };