Skip to content
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.

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

Uses prototype for declaring member functions #13

Merged
merged 3 commits into from
Mar 20, 2013
Merged

Uses prototype for declaring member functions #13

merged 3 commits into from
Mar 20, 2013

Conversation

baalexander
Copy link
Contributor

We had a mixed way of using ClassName.prototype.functionName = function() {} and this.functionName = function() {} (from inside the constructor). Using the prototype way allows for browser optimization as it can compile the functions once instead of having to compile each time an object is initialized.

@rctoris
Copy link
Contributor

rctoris commented Mar 20, 2013

I think in the future we should at the very least have the commits for the doc and build be separate so we can actually manage pull request reviews.

@baalexander
Copy link
Contributor Author

One issue I noticed when doing the build this time is it uses the person's home directory name in documentation generation, creating a much bigger diff.

@rctoris
Copy link
Contributor

rctoris commented Mar 20, 2013

Maybe once things are more stable we should just pull request in source changes.

rctoris added a commit that referenced this pull request Mar 20, 2013
Uses `prototype` for declaring member functions
@rctoris rctoris merged commit d1c3fae into RobotWebTools:groovy-devel Mar 20, 2013
@ghost ghost assigned rctoris May 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants