On 10/04/2013 10:01 PM, Nathaniel McCallum wrote:
On Fri, 2013-10-04 at 13:37 -0400, Nathaniel McCallum wrote:
This patch is preparatory for the OTP CLI patch.
I'm not quite sure why, but this patch apparently changes the output
of ./makeapi. This change is now included in the attached patch. Is this
a mistake of some sort? Or is this just correct?
Nathaniel
Apparently, Param.clone() uses all options, even those that are left out
of what API.txt reports (see __kw and __clonekw in Param.__init__). IMO
it's not ideal behavior, makeapi should have used __clonekw directly.
Anyway, to keep API.txt as before, only clone if necessary:
def __clone(self, param, **kw):
if 'optional_create' in param.flags:
kw['required'] = False
- return param.clone(**kw)
+ if kw:
+ return param.clone(**kw)
+ else:
+ return param
--
PetrĀ³
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel