The check for ListChanged in this method is a problem if you override OnListChanged because it won't get called.

protected virtual void OnEntityInListOnEntityContentsChanged(object sender, EventArgs e)
{
  if (this.ListChanged == null)
    return;
  this.OnListChanged(this.IndexOf((T) sender), ListChangedType.ItemChanged);
}

(A further issue is that the very act of looking at these fields in the debugger cause the DefaultView to be created which sets ListChanged and so allows the event to fire! Took me ages to work out why it was working in the debugger but not in real code)

Please provide more info, check the forum guidelines for more details.

Sorry - thought this was a general question.

Designer: v3.5 final Runtime: v3.5.12.0330 Adapter, .NET 4 SQL Server 2008

Given that the code calls this method...

    protected virtual void OnListChanged(int index, ListChangedType typeOfChange)
    {
      if (this._surpressListChangedEvents)
        return;
      if (typeOfChange == ListChangedType.Reset)
        this._entityIndices = (Dictionary<Guid, int>) null;
      if (this.ListChanged == null)
        return;
      this.ListChanged((object) this, new ListChangedEventArgs(typeOfChange, index));
    }

... and would be a NOP anyway, I am guessing this is a time-saver to prevent calls to this.IndexOf((T) sender when nothing is listening?

If you can confirm that that is the only reason, I can get around that by overriding OnEntityInListOnEntityContentsChanged to always make the call.

Confirmed.