[PATCH] Third and Final (hopefully) (was Re: [PATCH] First pass at finishing the last major feature on the TODO list (RequestName))

[Havoc Pennington]

Hi,

I looked at this patch in a less-thorough way than I wanted, but did
have a quick read finally. The only question I have is that in
bus_service_add_owner(), it looks like it's possible to replace 
the current head of the queue; but if you replace the existing owner
with a new owner here, is the "name lost" signal sent to the old owner?
Is it possible to even run this code or is swap_owner() used
instead? It's a little hard to see what's going on without applying the
patch and looking at the finished code.

Havoc