[ Quoting <notifications@github.com> in "Re: [skydns] Allow skydns itself to..." ]
> > @@ -184,10 +176,10 @@ func (c *Client) GetAllServicesDNS() ([]*msg.Service, error) {
> >  		if v, ok := r.(*dns.SRV); ok {
> >  			s[i] = &msg.Service{
> >  				// TODO(miek): uehh, stuff it in Name?
> > -				Name: v.Header().Name + " (Priority: " + strconv.Itoa(int(v.Priority)) + ", " + "Weight: " + strconv.Itoa(int(v.Weight)) +")",
> > +				Name: v.Header().Name + " (Priority: " + strconv.Itoa(int(v.Priority)) + ", " + "Weight: " + strconv.Itoa(int(v.Weight)) + ")",
> 
> While you are doing some refactoring do you think that `fmt.Sprintf` should be more readable for things like this? 

hadn't thought about using Sprintf here, but yes, your completely right.

But this should be done in another PR and we should talk about the output,
because right now it is put in the Name which is kinda of ugly.

-- 
   Miek Gieben