New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Don't initialize leaf namespace when rendering namespaces #1

Closed

ericf opened this issue May 3, 2013 · 0 comments · Fixed by #3

Assignees

Labels

Milestone

Collaborator

ericf commented May 3, 2013

Current, when rendering an exposed object's namespaces, the leaf namespaces are initialized as objects:

(function (g) {
// -- Namespaces --
g.YUI || (g.YUI = {});
g.YUI.Env || (g.YUI.Env = {});
g.YUI.Env.FOO || (g.YUI.Env.FOO = {});

// -- Exposed --
g.YUI.Env.FOO = "foo";
}(this));

This is unnecessary since the values to the leaf namespaces are assigned directly after the namespaces are initialized.

ghost assigned ericf

ericf mentioned this issue

Improve namespace rendering #3

Merged

ericf closed this as completed in

09696a7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment