[Freeipa-devel] [PATCH] 0194 Fix selector of protocol for LSA RPC binding string
Alexander Bokovoy
abokovoy at redhat.com
Thu Aug 6 11:34:23 UTC 2015
----- Original Message -----
>
>
> On 08/05/2015 08:40 PM, Alexander Bokovoy wrote:
> > Hi,
> >
> > attached patch fixes a bug
> > https://bugzilla.redhat.com/show_bug.cgi?id=1249455
> >
> > details are in the commit message.
> >
> >
> >
>
> Looks good to me, generates bindings strings as described in the BZ.
>
> Just a readability nitpick, can we get rid of the binding_template
> lambda abstraction and use something like this?
>
> binding_template=u'%s:%s[%s]'
> return [binding_template % (t, remote_host, o) for t in transports
> for o in options]
>
> or just plain:
>
> return [u'%s:%s[%s]' % (t, remote_host, o) for t in transports for o
> in options]
>
> instead of:
>
> binding_template=lambda x,y,z: u'%s:%s[%s]' % (x, y, z)
> return [binding_template(t, remote_host, o) for t in transports for
> o in options]
Up to you. :)
--
/ Alexander Bokovoy
More information about the Freeipa-devel
mailing list