Re: [PATCH] Make remote_host available in hook_pre_connection

2009-01-13 Thread David Nicol
On Mon, Jan 12, 2009 at 5:11 PM, Matt Sergeant wrote: > Except doesn't work with -async. Well, it works, but blocks. a deferred approach would help async, as the job to look up the remote name could get spawned at pre-connection time and it wouldn't have to block until it was needed, which might

Re: [PATCH] Deprecate DECLINED with NEXT

2009-01-13 Thread David Nicol
On Mon, Jan 12, 2009 at 2:40 PM, Jared Johnson wrote: > I can't say I agree with this patch, even though it would represent a less > significant change and satisfy more parties.. it would also make it more > likely for both terms to be used throughout the code, decreasing readability > and increas

Re: [PATCH] Make remote_host available in hook_pre_connection

2009-01-13 Thread Matt Sergeant
On Tue, 13 Jan 2009 08:38:09 -0600, David Nicol wrote: > On Mon, Jan 12, 2009 at 5:11 PM, Matt Sergeant wrote: >> Except doesn't work with -async. Well, it works, but blocks. > > a deferred approach would help async, as the job to look up the remote > name could get spawned at pre-connection time