[2626] in BarnOwl Developers

home help back first fref pref prev next nref lref last post

Re: [PATCH] Don't ignore the error code when doing a zwrite

daemon@ATHENA.MIT.EDU (Nelson Elhage)
Tue Sep 14 10:07:25 2010

From: Nelson Elhage <nelhage@MIT.EDU>
To: Jeffrey Hutzelman <jhutz@cmu.edu>, David Benjamin <davidben@mit.edu>,
        barnowl-dev@mit.edu
Cc: jhutz@cmu.edu
In-Reply-To: <EA29CB243E87318C98F351BF@minbar.fac.cs.cmu.edu>
Date: Tue, 14 Sep 2010 10:07:21 -0400

On Tue, 14 Sep 2010 09:45:48 -0400, Jeffrey Hutzelman <jhutz@cmu.edu> wrote:
> --On Monday, September 13, 2010 09:14:01 PM -0400 Nelson Elhage 
> <nelhage@mit.edu> wrote:
> 
> > Cool, definitely a good call.
> >
> > It's not hugely important, but is continuing to try further recipients
> > really the right plan if one fails? I feel like I'd expect it to bail
> > out after the first error.
> 
> So if I send to five recipients and the third one isn't subscribed, the 
> first two get my zgram and the other two don't?  Or am I misinterpreting 
> how much work send_zephyr() actually does?

send_zephyr just does a ZSrvSendNotice (... no, I'm not sure why we need
to do that instead of just ZSendNotice), so I believe that if they
aren't subscribed, you won't find out until you get a ZSRVACK_NOTSENT
back from the server, which will get processed later by the mainloop.

I guess I should probably verify that before pushing this to a release,
but that's my understanding.

- Nelson

> 
> 

home help back first fref pref prev next nref lref last post