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

Allow failure on glutin platform and minor nits #33

Merged
merged 6 commits into from Oct 29, 2015
Merged

Allow failure on glutin platform and minor nits #33

merged 6 commits into from Oct 29, 2015

Conversation

@emilio
Copy link
Member

emilio commented Oct 15, 2015

We should probably change Err(&str) to another type of error...

@emilio
Copy link
Member Author

emilio commented Oct 15, 2015

May you take a quick look at this and tell me if it compiles fine on windows @vvuk? :)

(It definitely should, but I have no way to easily test it actually, and our CI is broken because of aster)

@emilio emilio force-pushed the windows-tidy branch 2 times, most recently from a682507 to 1519437 Oct 28, 2015
emilio added 4 commits Oct 15, 2015
We should probably change `Err(&str)` to another type of error...
Also implements the missing methods.

Don't know when it was removed on the first place.
@emilio emilio force-pushed the windows-tidy branch from 1519437 to de3e79a Oct 29, 2015
emilio added 2 commits Oct 29, 2015
This commit updates the tests to the newest rust-layers.
@emilio emilio force-pushed the windows-tidy branch from 710c520 to 4b02652 Oct 29, 2015
@emilio
Copy link
Member Author

emilio commented Oct 29, 2015

Seems like CI is broken again due to aster.

I could confirm the build and tests passing on linux and mac, and tested the code used in windows and unimplemented platforms.

I also changed the feature test, that was working previously just because I was lucky and the vector I was creating contained the memory from old data (I don't even know how I wanted back in the day to get the texture pixels with TexImage2D instead of GetTexImage).

Android-specific code shouldn't be affected here, so I'm going to merge this.

emilio added a commit that referenced this pull request Oct 29, 2015
@emilio emilio merged commit 05a1ea3 into master Oct 29, 2015
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
@jdm
Copy link
Member

jdm commented Oct 29, 2015

Good find on the feature test problem :)

@emilio emilio deleted the windows-tidy branch Nov 1, 2015
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

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