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

Use unique names for private storage fields in Vector(2,3,4), Matrix(2,3,4) #131

Merged
merged 1 commit into from
Apr 13, 2015
Merged

Conversation

kevmoo
Copy link
Collaborator

@kevmoo kevmoo commented Apr 13, 2015

Fixes #126

@Fox32
Copy link
Contributor

Fox32 commented Apr 13, 2015

Can we do #132 first and then redo this one? Some code that was later added uses the public getter instead of the private variables

@kevmoo
Copy link
Collaborator Author

kevmoo commented Apr 13, 2015

@Fox32 Done. Rebased

@Fox32
Copy link
Contributor

Fox32 commented Apr 13, 2015

Looks good, but misses vector_math_64

@kevmoo
Copy link
Collaborator Author

kevmoo commented Apr 13, 2015

Ooops. Fixed. Thanks.

@Fox32 Fox32 mentioned this pull request Apr 13, 2015
johnmccutchan added a commit that referenced this pull request Apr 13, 2015
Use unique names for private storage fields in Vector(2,3,4), Matrix(2,3,4)
@johnmccutchan johnmccutchan merged commit 0155dc3 into google:master Apr 13, 2015
@kevmoo kevmoo deleted the unique_storage branch April 13, 2015 21:33
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.

Rename _storage private field to be different in each class
3 participants