|
Boost : |
Subject: Re: [boost] [config] any macro to detect "Rvalue references for *this"?
From: Beman Dawes (bdawes_at_[hidden])
Date: 2014-05-27 08:54:04
On Mon, May 26, 2014 at 5:04 AM, Andrzej Krzemienski <akrzemi1_at_[hidden]>wrote:
> 2014-05-26 10:32 GMT+02:00 John Maddock <boost.regex_at_[hidden]>:
>
> > Thanks for pointing me to these instructions. However, it may require of
> me
> >> more time that I am can afford. I do not understand some of the
> >> instructions. One of the bullets says:
> >>
> >> "cd into libs/config/test/all and run bjam *MACRONAME compiler-list*,
> >> where
> >>
> >>> *MACRONAME* is the name of the new macro, and *compiler-list* is a
> space
> >>>
> >>> separated list of compilers to test with."
> >>>
> >>>
> >> This "compiler-list", I obviously want the macro to work for all
> possible
> >> compilers supported by Boost (and perhaps more), but locally, I can only
> >> test it on one. In order to define macro correctly, I would have to use
> >> Boost's testing facility, but they are only run on develop and master
> >> branches which I cannot commit to. Without having tested my macros on
> all
> >> compilers, I am not sure if my pull request would be a decent one. IOW,
> I
> >> have a chicken and egg problem here, which I do not know how to solve.
> >>
> >> any help would be much appreciated.
> >>
> >
> > If you can provide the test case I will test locally before committing
> > anyway.
>
>
> Thanks for the reply. What is expected of me then? Only the test case in
> the ipp file? Or also bjam generated test files? Or also the macro
> definitions in the headers, and the docs?
>
>
The more you do the less the library maintainer has to do, and that speeds
processing the pull request. Most of the above is pretty trivial.
For example, to update the docs, edit libs/config/doc/macro_reference.qbk
with your favorite text editor. Following the lines that read:
[[`BOOST_NO_CXX11_RVALUE_REFERENCES`][The compiler does not support
r-value references.
]]
Add something like:
[[`BOOST_NO_CXX11_RVALUE_REF_THIS`][The compiler does not support
r-value references for *this .
]]
One thing that can be a bit tricky is editing the compiler files. You might
want to just do the one you are testing, and leave the others to config
maintainers.
--Beman
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk