-
Notifications
You must be signed in to change notification settings - Fork 63
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
#pragma warning #859
Comments
A code example demonstrating what you mean might help |
At the moment I am using different names: But after I noticed the possibility of overloading, I want to change it to one name, which will differ only in the number of parameters |
I don't think macros are getting any additional support--bail would remove them given the chance. I'm not sure overloaded macros are supposed to exist. I also can't think of any other use case where suppressing warnings is a good idea for SP. Usually its a strong indicator of problems in the code unlike in C/C++ where it's not uncommon to run into warnings that you just have to live with because the codebase is built around them. |
Yes, you seem to be right. I recently tried to make 3 macros with the same name, and for some reason it was destroyed, although with 2 macros with the same name everything works fine. It was very strange. Tested on spider.limetech.org |
Can you show an example of what you're looking for here? |
I make include with macros that use an overload (which oddly enough in SP works fine) which results in a 201 warning, so I suggest adding a
#pragma warning
like in msvchttps://learn.microsoft.com/en-us/cpp/preprocessor/warning?view=msvc-170
The text was updated successfully, but these errors were encountered: