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

Update for android port. #66

Closed
wants to merge 2 commits into from
Closed

Update for android port. #66

wants to merge 2 commits into from

Conversation

@recrack
Copy link
Contributor

recrack commented Dec 19, 2013

This PR is for android port.

@metajack
Copy link
Contributor

metajack commented Dec 19, 2013

Looks fine except for the inconsistent names.

@aydinkim
Copy link

aydinkim commented Dec 20, 2013

Recrack, you need to fix the naming rules have more consistency.

@aydinkim
Copy link

aydinkim commented Dec 26, 2013

@metajack It seems that Recrack fixed inconsistent name on 3901d58

@@ -365,6 +365,16 @@ pub type GLintptr = intptr_t;

pub type GLsizeiptr = ssize_t;

// GL info
pub static GL_VENDOR: c_uint = 0x1F00;

This comment has been minimized.

Copy link
@metajack

metajack Dec 27, 2013

Contributor

These need to omit the GL_ prefix like the other statics.

@metajack
Copy link
Contributor

metajack commented Dec 27, 2013

r+ after the statics get renamed.

@aydinkim
Copy link

aydinkim commented Dec 27, 2013

Recrack is on a leave. I'll do that instead of him.

@aydinkim
Copy link

aydinkim commented Dec 27, 2013

It seems
+pub static GL_VENDOR: c_uint = 0x1F00;
+pub static GL_RENDERER: c_uint = 0x1F01;
+pub static GL_VERSION: c_uint = 0x1F02;
+pub static GL_EXTENSIONS: c_uint = 0x1F03;
+pub static GL_COLOR_BUFFER_BIT: c_uint = 0x00004000;

is duplicated.
I will eliminate that changes and then do r+ on from the another PR base because I can not access recrack's personal repo.

@aydinkim aydinkim mentioned this pull request Dec 27, 2013
@aydinkim
Copy link

aydinkim commented Dec 27, 2013

Instead if this PR,
I committed #67

@metajack metajack closed this Dec 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.