[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