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,


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

Reply via email to