[Chandler-dev] Sharing API for edit/update
Morgen Sagen
morgen at osafoundation.org
Tue Jan 30 14:01:31 PST 2007
On Jan 30, 2007, at 1:21 PM, Phillip J. Eby wrote:
> At 12:37 PM 1/30/2007 -0800, Morgen Sagen wrote:
>> Additionally, If the item version
>> embedded in the EIMML is less than or equal to the version number
>> last seen from this peer, an OutOfSequence exception is raised.
>
> Just an FYI, but this isn't really an error condition. If we
> receive an out-of-sequence email, we should simply ignore it.
I thought about just ignoring it, but then I thought perhaps the
caller might want to be aware of this situation, and possibly alert
the user(?). Not sure.
>
> (By the way, nice work on the implementation; I see the doctests
> got a lot shorter and clearer after the refactoring to make states
> handle their own merging.)
>
I neglected to give kudos to Phillip for his elegant merge algorithm
and many helpful suggestions. Thanks, pje!
~morgen
More information about the chandler-dev
mailing list