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

Adds texture attribute to bullet #8

Closed
wants to merge 3 commits into from
Closed

Adds texture attribute to bullet #8

wants to merge 3 commits into from

Conversation

robdit
Copy link

@robdit robdit commented Oct 5, 2018

Fixes #4, I didn't implement the texture caching because I don't know how. Otherwise this lets you specify the filepath where the texture will be from loaded. To make use of this new feature you'll have to modify the danmaku.cpp.

@hanss314
Copy link
Owner

hanss314 commented Oct 5, 2018

Since everything in the group will always have the same texture, could you just store the texture in the group? Also rather than having to load the image from file every frame, just store the bytes in the group

@robdit
Copy link
Author

robdit commented Oct 6, 2018

When I put the image loading right before glTexImage2D(..) it correctly loads the image with colors. Elsewhere it turns it into a black square, any ideas why?

@hanss314 hanss314 closed this Nov 1, 2018
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