Code review comment for lp:~thumper/nux/properties

Revision history for this message
Neil J. Patel (njpatel) wrote :

The owner would emit the signal when they change the underlying store. As long as its possible to emit the changed signal, that should cover the use case that Mikkel mentioned.

Sent from my iPhone

On 11 Jul 2011, at 10:36, Tim Penhey <email address hidden> wrote:

> On Mon, 11 Jul 2011 19:31:21 you wrote:
>> Had a quick skim - nice work Tim :-) I have two points I want to raise:
>> a) If I understand correct the default behaviour for a ROProperty is to
>> not signal on change. I think this is this is the wrong behaviour for two
>> reasons: 1) From my experience RO properties are used for things like a
>> bool "connected" or "mapped" where they reflect unchangeable aspects of
>> the underlying system. Normally you'll want change notifications on those.
> My only real issue with ROProperty is that we don't know if it has changed or
> not. So how do we know to signal?
> --
> Your team Unity Team is requested to review the proposed merge of lp:~thumper/nux/properties into lp:nux.

« Back to merge proposal