detritus posted on Tuesday, September 30, 2008

It's kind of small thing but I think try block should start right after the line

this.RaiseListChangedEvents = false;

For example if I call the SaveItem method manually and gave it an index that not in the list, it'll raise an exception on the line

T item = this[index];

and will not be able to recover the property RaiseListChangedEvents and _activelySaving variable because the line is not protected (by try block)

Sinan

RockfordLhotka replied on Wednesday, October 01, 2008

Thank you for reporting this - I'll add it to the task list.

sergeyb replied on Monday, October 06, 2008

The issue has been corrected in 3.6 and 3.5 (VB and C#). Unit tests for this situation have been added in all version above.

Thanks.