Review: code (2/2)

[iain-anderson]

• The time taken in the main loop (call to UA_sleep_ms) should be configurable with an entry in the [Driver] config section

• Also in this loop, the releases and re-acquisitions of the mutex could allow the structure to be changed while we are iterating it. The mutex should be held for the whole pass. If the calls to UA_Client_runAsync take significant time, create a new list of clients which require the call, and iterate this outside of the mutexed section.

• opcua_driver.conn_length and ua_conn_addr_status.length should be unsigned. Use iterator variables of the same type in for() loops which reference them