Hi JB,

just tripped over ArrayDimension and noticed its a mutable value type.
LowerBound and UpperBound are get/set while the set properties are never
used. I propose removing the set accessor for two reasons:

1. They are not used anywhere
2. And even if someone would, it's probably not doing what he thinks.

I think the last one is self explaining and it's universally agreed that
mutable value types should be avoided.

Regards,
Johannes

-- 
--
mono-cecil

Reply via email to