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

MSVC2015 - warning 6326 - Potential comparison of a constant with another constant. #263

Closed
omascia opened this issue Jan 11, 2016 · 4 comments

Comments

@omascia
Copy link

omascia commented Jan 11, 2016

I think the linked picture says it all. I have a 6326 reported on line 1788 of format.h (2.0). On that line I see bool fm:internal::check(bool value) whose intent is to suppress bogus "conditional expression is constant". But it looks like "potential comparison of a constant with another constant" is emitted instead.
The attached partial print-screen shows the warning reported and the format.h line 1788 in context.
w6326

@vitaut
Copy link
Contributor

vitaut commented Jan 11, 2016

Thanks for reporting this. The make_arg_list function has been dropped from the current development version (master branch) so you should no longer get this warning.

@omascia
Copy link
Author

omascia commented Jan 11, 2016

It is my understanding that I can for now safely ignore this warning. Am I right? (I'll test the master branch separately, but I'd like to release some code based on the 'released' code in between). Safe according to you?

@vitaut
Copy link
Contributor

vitaut commented Jan 11, 2016

This is a completely useless (but annoying) warning, you can safely ignore it or suppress with a pragma.

@vitaut
Copy link
Contributor

vitaut commented Jan 11, 2016

Since there is no make_arg_list any more I'm going to close this, but feel free to reopen (or open another issue) if you get a similar warning elsewhere.

@vitaut vitaut closed this as completed Jan 11, 2016
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

2 participants