Steve Loranz wrote:
My main concern here is that this refactoring doesn't take into
account properties that only need get or get/set accessors instead of
full access. For the contributor that isn't thinking about it, this
may open up unintentional use of the properties down the line. Quite
honestly, my early use of properties in image-factory could stand some
cleanup in terms of read, read/write, read/write/delete and I worry
that hiding that will only exacerbate the issue.
Mark and I chatted about refining the prop code to allow for
specifying the accessors you want a property to provide. We agreed to
move forward with the current prop code and add a task to add the
accessor specifiers.
so, ACK
Thanks.