C#

Bernhard Hiller wrote:

C#

Task task = new Task(() => RemoteChannelProvider.Call(_channel => _channel.SetInfraredFocus(_clientName, _focus)));

1. SetInfraredFocus returns a Task;
2. Call may or may not discard that task;
3. Even if it doesn't discard the task, the new Task constructor definitely discards it - none of the overloads accept a Func<Task>;
4. The task returned from the client therefore doesn't observe the result of the task returned from the server. It could even complete before the server call has finished.

public Task SetInfraredFocus(string _clientName, double _focus)
{
    return Task.Run(() => RemoteChannelProvider.Call(_channel => _channel.SetInfraredFocus(_clientName, _focus)));
}

public delegate void CallProxyDelegate<in T>(T _channel);
...
void Call(CallProxyDelegate<T> _codeBlock, [CallerMemberName] string _functionName = "");

m_ConfigurationChannelFactory = new ConfigurationDuplexChannelFactory<T>(m_CallbackObject, ConfigurationName, null, config);
...
m_CommunicationChannel = (ICommunicationObject)m_ConfigurationChannelFactory.CreateChannel();
...
_codeBlock((T)m_CommunicationChannel);

public Task SetInfraredFocus(string _clientName, double _focus)
{
    return Task.Run(delegate
    {
        Task result = null;
        RemoteChannelProvider.Call(_channel => result = _channel.SetInfraredFocus(_clientName, _focus));
        return result;
    });
}