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

[[nodiscard]] for ntsa::Error #50

Open
smtrfnv opened this issue Jun 9, 2023 · 0 comments
Open

[[nodiscard]] for ntsa::Error #50

smtrfnv opened this issue Jun 9, 2023 · 0 comments

Comments

@smtrfnv
Copy link
Contributor

smtrfnv commented Jun 9, 2023

I was thinking if it can be useful for us to declare ntsa::Error as class [[nodiscard]] ntsa::Error when the compiler supports this feature.

At the moment such declaration results in a lot of compilation warnings, mainly in test drivers. But as the idea of ntsa::Error is to return an error status from a function it's better to not ignore 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

No branches or pull requests

1 participant