don't do a pending save if slave is syncing.
authorRobin Gareus <robin@gareus.org>
Tue, 16 Oct 2012 20:53:08 +0000 (20:53 +0000)
committerRobin Gareus <robin@gareus.org>
Tue, 16 Oct 2012 20:53:08 +0000 (20:53 +0000)
commit625ee55df0d84e708ce269ee47cc1e36881cd71e
treef7715934741730b3049ce9beba57834ffcdbe458
parent4c9fcd407c0333892a38c97a9598b1a9dd3ca2a4
don't do a pending save if slave is syncing.

13:00 < rgareus> When a slave does initial sync, it sets speed=0, pos=XXX (required by session_process.cc state-machine to init)
13:01 < rgareus> This triggers a locate(roll=false) which in turn triggers a realtime_stop() which in turn triggers a non_realtime_stop().
[..]
13:06 < rgareus> las: the problem I have with non_realtime_stop() is that it does save a pending state IFF  get_record_enabled() is true.
13:06 < rgareus> The save can take ages (seconds), which will void the initial sync of the slave.
13:07 < rgareus> The slave enters a live-lock: sync, save, re-sync, save...
13:07 < las> rgareus: understood
13:07 < rgareus> las: I propose to workaround this: only save pending state if there is no slave or the slave is not locked.
13:07 < las> rgareus: another reason why recording + slave == bad idea :(
13:07 < las> rgareus: but yes, that sounds fine to me
13:07 < rgareus> AFICT this is not harmful. It only affects pre-record settings.
13:07 < rgareus> 'did_record' is used to save a full state after each recording.

git-svn-id: svn://localhost/ardour2/branches/3.0@13288 d708f5d6-7413-0410-9779-e7cbd77b26cf
libs/ardour/session_transport.cc