8000 GLM 1.0 compatibility by cjmayo · Pull Request #721 · elalish/manifold · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

GLM 1.0 compatibility #721

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 occasional 8000 ly send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 28, 2024
Merged

GLM 1.0 compatibility #721

merged 1 commit into from
Jan 28, 2024

Conversation

cjmayo
Copy link
Contributor
@cjmayo cjmayo commented Jan 27, 2024

Using glm/gtx/compatibility.hpp without GLM_ENABLE_EXPERIMENTAL has become an error:

error: #error "GLM: GLM_GTX_compatibility is an experimental extension
               and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL
	       before including it, if you really want to use it."

Became an error in:
g-truc/glm@f86092a#diff-b25c43cdda0d7be5dc5ea91beefb168cc1bd2522d60c7aa9b6c6164c0630e173

I haven't noticed any other effects from defining GLM_ENABLE_EXPERIMENTAL. But...

Using glm/gtx/compatibility.hpp without GLM_ENABLE_EXPERIMENTAL has
become an error:

error: #error "GLM: GLM_GTX_compatibility is an experimental extension
               and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL
	       before including it, if you really want to use it."
Copy link
codecov bot commented Jan 27, 2024
< 8000 /h3>

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9690305) 91.67% compared to head (59f4e6c) 91.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #721   +/-   ##
=======================================
  Coverage   91.67%   91.67%           
=======================================
  Files          37       37           
  Lines        4732     4732           
=======================================
  Hits         4338     4338           
  Misses        394      394           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pca006132
Copy link
Collaborator

Thanks!

@pca006132 pca006132 merged commit 0b7ee87 into elalish:master Jan 28, 2024
@cjmayo cjmayo deleted the glm-1.0 branch January 29, 2024 20:02
cartesian-theatrics pushed a commit to cartesian-theatrics/manifold that referenced this pull request Mar 11, 2024
Using glm/gtx/compatibility.hpp without GLM_ENABLE_EXPERIMENTAL has
become an error:

error: #error "GLM: GLM_GTX_compatibility is an experimental extension
               and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL
	       before including it, if you really want to use it."
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
0