forked from icaven/glm
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
No unused function result warning #1217
Comments
christophe-lunarg
added
enhancement
OS: Platform independent
and removed
PR requested
labels
Feb 8, 2024
I like this feature request, I'll try to implement it someday soon (TM)! |
christophe-lunarg
added a commit
that referenced
this issue
Feb 8, 2024
That was quick, thanks a ton for awesome work |
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 9, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
It's probably not exaustive but future contributions would be welcome! |
christophe-lunarg
added a commit
that referenced
this issue
Feb 10, 2024
Zuzu-Typ
pushed a commit
to Zuzu-Typ/glm
that referenced
this issue
Oct 11, 2024
Zuzu-Typ
added a commit
to Zuzu-Typ/glm
that referenced
this issue
Oct 11, 2024
This reverts commit 1bc816e.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm using glm::rotate, and if I forget to use the function result somewhere in my code, the error is silent, the results will be wrong,
marking glm::rotate as [[nodiscard]] fixes it, but that needs to be done for all similar functions
The text was updated successfully, but these errors were encountered: