In the routine updateDelay, I notice a local variable named slave_to_master_delay is calculated and used with maxDelay, but is never stored. Instead, the delaySM seems to be used thereafter. I'm not sure if this is a bug or just some old code that isn't important. please advise.
Sorry, I should have mentioned I am in the v2 branch, rev 138
Sorry to take so long. The variable is used to effect a local check and if that check fails then delaySM is not updated. Basically it protects delaySM in certain error cases. This is not a bug.