smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-06 14:54:56 +01:00
Code Issues Projects Releases Wiki Activity

temporal: minor comment fixes

Browse source 
Some typos and copy editing to remove a (pretty much) duplicated chunk.
This commit is contained in:
Mads Kiilerich 2022-11-17 15:26:17 +01:00 • committed by Paul Davis
parent 69c5c6e1e8
commit 368f0aec44
2 changed files with 3 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

2
libs/temporal/temporal/tempo.h
View file

@ -735,7 +735,7 @@ class /*LIBTEMPORAL_API*/ TempoMap : public PBD::StatefulDestructible
* one could hold the map for, etc. * one could hold the map for, etc.
* *
* Elsewhere in the codebase, we use RCU to solve this sort of * Elsewhere in the codebase, we use RCU to solve this sort of
* issue. For example, if we need to operate an the current list of * issue. For example, if we need to operate on the current list of
* Routes, we get read-only copy of the list, and iterate over it, * Routes, we get read-only copy of the list, and iterate over it,
* knowing that even if the canonical version is being changed, the * knowing that even if the canonical version is being changed, the
* copy we are using will not. * copy we are using will not.

9
libs/temporal/temporal/timeline.h
View file

@ -110,7 +110,7 @@ class LIBTEMPORAL_API timepos_t : public int62_t {
/* don't provide operator+(samplepos_t) or operator+(superclock_t) /* don't provide operator+(samplepos_t) or operator+(superclock_t)
* because the compiler can't disambiguate them and neither can we. * because the compiler can't disambiguate them and neither can we.
* to add such types, create a timepo_t and then add that. * to add such types, create a timepos_t and then add that.
*/ */
/* operator-() poses severe and thorny problems for a class that represents position on a timeline. /* operator-() poses severe and thorny problems for a class that represents position on a timeline.
@ -239,11 +239,6 @@ class LIBTEMPORAL_API timepos_t : public int62_t {
explicit timepos_t (bool b, int64_t v) : int62_t (b, v) {} explicit timepos_t (bool b, int64_t v) : int62_t (b, v) {}
explicit timepos_t (int62_t const & v) : int62_t (v) {} explicit timepos_t (int62_t const & v) : int62_t (v) {}
/* these can only be called after verifying that the time domain does
* not match the relevant one i.e. call _beats() to get a Beats value
* when this is using the audio time domain
*/
/* these three methods are to be called ONLY when we have already that /* these three methods are to be called ONLY when we have already that
* the time domain of this timepos_t does not match the desired return * the time domain of this timepos_t does not match the desired return
* type, and so we will need to go to the tempo map to convert * type, and so we will need to go to the tempo map to convert
@ -306,7 +301,7 @@ class LIBTEMPORAL_API timecnt_t {
timecnt_t (TimeDomain td) : _distance (td != AudioTime, 0), _position (td) {} timecnt_t (TimeDomain td) : _distance (td != AudioTime, 0), _position (td) {}
timecnt_t (timecnt_t const &other) : _distance (other.distance()), _position (other.position()) {} timecnt_t (timecnt_t const &other) : _distance (other.distance()), _position (other.position()) {}
/* construct from sample count (position doesn't matter due to linear nature * of audio time */ /* construct from sample count (position doesn't matter due to linear nature of audio time) */
explicit timecnt_t (samplepos_t s, timepos_t const & pos); explicit timecnt_t (samplepos_t s, timepos_t const & pos);
explicit timecnt_t (samplepos_t s); explicit timecnt_t (samplepos_t s);