Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Remove use of deprecated macros in documentation#3

Open
mclow wants to merge 1 commit intoboostorg:developboostorg/local_function:developfrom
mclow:developmclow/local_function:developCopy head branch name to clipboard
Open

Remove use of deprecated macros in documentation#3
mclow wants to merge 1 commit intoboostorg:developboostorg/local_function:developfrom
mclow:developmclow/local_function:developCopy head branch name to clipboard

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 notifications@github.com
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 notifications@github.com
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.