Is this doc comment right? sync_finish() *starts* the sync?