Skip to content

Remove use of deprecated macros in documentation #3

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

mclow
Copy link
Contributor

@mclow mclow commented Sep 11, 2015

Replace uses in the documentation of the deprecated macro BOOST_NO_LOCAL_CLASS_TEMPLATE_PARAMETERS with BOOST_NO_CXX11_LOCAL_CLASS_TEMPLATE_PARAMETERS instead.

The code already uses BOOST_NO_CXX11_LOCAL_CLASS_TEMPLATE_PARAMETERS, this is just a documentation change.

I didn't touch the HTML files; I assume that they need to be regenerated from the QBK and checked in separately.

@mclow
Copy link
Contributor Author

mclow commented Oct 13, 2015

Ping?

@lcaminiti
Copy link
Member

Hi Marshall,

Yes, QuickBook needs to be ran to re-generate the HTML and those need to be
checked in Git. Also this macro should be changed if used in example/* and
test/* code. The same of the NO_LAMBDA macro you mentioned before and for
Boost.ScopeExit as well.

I am hoping to have time to do this in the next couple of weeks (as I
mentioned to you before, I was hoping to have some time early in Oct but I
did not find time to do that yet).

These are trivial changes to make... but I have been completely busy with a
project a work for the last few month plus I have just moved to a new
house. The work project paused last week and should not pick up again until
next week so I will try to make these changes by then.

Thanks.
--Lorenzo

On Tue, Oct 13, 2015 at 9:39 AM, Marshall Clow [email protected]
wrote:

Ping?


Reply to this email directly or view it on GitHub
#3 (comment)
.

--Lorenzo

@lcaminiti
Copy link
Member

Hi Marshall,

I have updated LocalFunction HTML docs. Do you want me to just commit those
to master? (I also updated ScopeExit's HTML docs.)

Thanks,
--Lorenzo

On Tue, Oct 13, 2015 at 9:39 AM, Marshall Clow [email protected]
wrote:

Ping?


Reply to this email directly or view it on GitHub
#3 (comment)
.

--Lorenzo

@mclow
Copy link
Contributor Author

mclow commented Oct 19, 2015

That would be great, thanks!

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

Successfully merging this pull request may close these issues.

2 participants