Weave calculated property notifications as if-else-if, not consecutive ifs about catel.fody

[GeertvanHorrik]

I am not sure how the weaver works when there are multiple properties, e.g.:

public int Y
{
    get { return X + B; }
}

public int Z
{
    get { return X + B; }
}

Will Catel.Fody correctly weave the change notifications for both Y and Z in a single if statement?

[bigworld12]

oh i didn't think of this use case
it should weave them into a single if statement, i think

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[GeertvanHorrik]

At this stage, I think it's too complex to implement where I don't see direct benefit. I did implement RaisePropertyChanged(() => property) yesterday though

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.