Code Inspection: Explicit delegate creation expression is redundant

tip

You can suppress this inspection to ignore specific issues, change its severity level to make the issues less or more noticeable, or disable it altogether.

Redundant delegate constructor calls may occur, for example, as a result of adding event handlers:

MyHandler += new EventHandler(MyMethod);

Below, the type of a delegate variable ValueChanged can be inferred from its declaration. To make code shorter, JetBrains Rider suggests removing a redundant explicit delegate’s constructor call:

internal class EventTest
{
    protected event EventHandler ValueChanged;
    protected EventTest()
    {
        ValueChanged += new EventHandler(HandleValueChanged);
    }
    private void HandleValueChanged(object sender, EventArgs e)
    {
        // ...
    }
    protected virtual void OnMyValueChanged()
    {
        ValueChanged?.Invoke(this, EventArgs.Empty);
    }
}

internal class EventTest
{
    protected event EventHandler ValueChanged;
    protected EventTest()
    {
        ValueChanged += HandleValueChanged;
    }
    private void HandleValueChanged(object sender, EventArgs e)
    {
        // ...
    }
    protected virtual void OnMyValueChanged()
    {
        ValueChanged?.Invoke(this, EventArgs.Empty);
    }
}