Merge pull request #162 from ChimeraTK/wip/killenb-DeviceAccess137_preWrite_doPreWrite_versionNumbers

Adapted to DeviceAccess API changes from DA#137

ExceptionHandlingDecorator: doReadTransferNonBlocking() and doReadTransferLatest() must always return true because there will be new data. The recovery in postRead() takes care of it.

was already removed for the ScalarAccessor and is tested at least there

This reverts commit 4f2b35be.

Was actually used and tested.

this was never really a problem, since it was always used with the BidirectionalProcessArray() which implements writeDestructively same as write

It already exists for ArrayAccessor::readAsync()...

transform logic_errors in doXxxTransferYyy that are already checked in the preXxx functions into asserts

If someone includes the Logger into an ApplcationModule and calls writeAll() the Logger variable message will be written. To catch that cases set message empty after calling sendMessage and ignore empty messages in the LoggingModule.

ExceptionHandlingDecorator: With the current (=old) the owner has to be informed about data invalidity as soon as the exception is reported, and then again after recovery.

Fixes #153

improved places where DeviceModule changes testableMode_deviceInitialisationCounter. Now matches with deviceHasError.

Now the writing in the DeviceModule is working.

Fixes #159

Merge iss130 branch into master
Closes #130 

- removed wrong waitForRecovery()
- some code improvements
- fixed wrong logic in doReadTransferAsync