Ok, interesting.  I mostly change that piece so that the naming was consistent.  When would nsqd actually use this field rather then the address supplied by --lookupd-tcp-address?  I've actually already got this branch running in production and it works fine (of course I have a single usecase and haven't tried many other configurations).  That said, should I go ahead and add the broadcast-address flag into the nsqlookup?