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

[solder_terrain_edge] rewrite of #954 #1542

Closed
wants to merge 20 commits into from
Closed

[solder_terrain_edge] rewrite of #954 #1542

wants to merge 20 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 4, 2016

Adds OpenGL support, fixed tabs vs space, avoid standard c++ container, using T3D native types.

@dottools
Copy link

dottools commented Mar 4, 2016

What's up with all the file renames? Think you're going to have to take your PR as a patch on top of a cleanly cloned Development branch and apply only the source code changes with said patch. Cause something went weird with commit tracking on these changes.

@ghost
Copy link
Author

ghost commented Mar 5, 2016

@dottools Sorry for confusion, wasn't my intention to open PR in that early stage (I unintentionally created it from the git client). I still was working on it...
However, now it should be ok.
It just needs a final review.
Thank you (:

@ghost ghost added the New feature label Mar 5, 2016
@ghost ghost added this to the 3.9 milestone Mar 5, 2016
@ghost ghost added the Final review label Mar 5, 2016
@ghost ghost self-assigned this Mar 5, 2016
@ghost ghost closed this Mar 5, 2016
@ghost ghost deleted the pr/954 branch March 5, 2016 16:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants