ValueTypeArrays should provide element names similar to row names for matrix types

[mkommend]

Note: The StringArray class must also be adapted accordingly.

[mkommend]

r9657: Added element names for the ValueType- and StringArray classes.

[mkommend]

r9692: Corrected changing of element names if a ValueType- or StringArray is resized.

Edited by abeham: fixed revision

[abeham]

I reviewed r9657:

• I would not expose the property as IEnumerable<string>, but as something that can also be accessed by index. If the array members can be accessed by index, so should their names. Why not string[]?
• StringConvertibleArrayView: Does setting each of the HeaderCell.Value properties fire a paint event?
• StringArray.ElementNames_set: Throw InvalidOperationException instead of NotSupportedException when the state of the object is ReadOnly, see this ​remark.

Reviewed r9692:

• Do we need the "Element" prefix for each element name by default?

[mkommend]

r9695: Removed default element names for ValueTypeArrays and added more efficient implementation for the row header update in the view.

[mkommend]

Replying to abeham:

• I would not expose the property as IEnumerable<string>, but as something that can also be accessed by index. If the array members can be accessed by index, so should their names. Why not string[]?

The type cannot be changed to string[], because I would not get any change notifications. Maybe IList<string> with an underlying ObserveableList could be a solution. However, the performance of the row header update was improved and I am fine with the enumerable for the moment.

• StringConvertibleArrayView: Does setting each of the HeaderCell.Value properties fire a paint event?

I don't know.

• StringArray.ElementNames_set: Throw InvalidOperationException instead of NotSupportedException when the state of the object is ReadOnly, see this ​remark.

NotSupportException was chosen to stay consistent with all other HL data types and properties (e.g., Length).

Reviewed r9692:

• Do we need the "Element" prefix for each element name by default?

This was addressed with r9695.

[abeham]

I reviewed r9695. The changes are okay.

[mkommend]

r9737: Integrated the performed changes in the stable branch.