-
Notifications
You must be signed in to change notification settings - Fork 50
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
List of non BOOST-prefixed macros #52
Comments
I have not seen any serious discussion of the requirement for prefixing all macros with Boost on the mailing list. Could you give me a link to the email thread, please? |
@hkaiser sounds from the related discussion that this is a good thing to do even for the include guards. Shouldn't be hard to tackle. Should we simply prepend |
@jefftrull Just prepending |
There is one remaining macro in the list above, from Phoenix - we would have to coordinate with them boostorg/phoenix#90 |
I just realized that the remaining Phoenix macros are from Spirit Classic and (judging from a quick look at their issue tracker) are unlikely to be addressed. At this late date it would be better to upgrade to V2 or X3 than to pursue the last few macros. |
The following macros are missing a BOOST_ prefix, which is against Boost library guidelines. I have logged the Phoenix and Spirit issues separately at boostorg/phoenix#90 and boostorg/spirit#562:
The text was updated successfully, but these errors were encountered: