On 11/3/2010 9:10 AM, Adam Young wrote:
The IPA and ipa_cmd are defined twice.

+var IPA;
+var ipa_cmd;

+var IPA = ( function () {
function ipa_cmd(name, args, options, win_callback, fail_callback,
objname)

Fixed

The duplicate IPA declaration is fixed, but the ipa_cmd is still
declared twice:

var ipa_cmd;

function ipa_cmd( ... ) { ... }

Is var ipa_cmd really necessary? We could also do this if we have to:

var ipa_cmd;

ipa_cmd = function( ... ) { ... };

But is there anything wrong with just the plain function declaration?

--
Endi S. Dewata

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to