Subject: Re: [boost] Standalone boost::ignore_unused_variable_warning()
From: Andrey Semashev (andrey.semashev_at_[hidden])
Date: 2014-05-19 10:48:12
On Monday 19 May 2014 16:40:05 Adam Wulkiewicz wrote:
> 2014-05-19 15:00 GMT+02:00 Peter Dimov <lists_at_[hidden]>:
> > Adam Wulkiewicz wrote:
> >> I requested a pull for unused_variable:
> >> https://github.com/boostorg/utility/pull/8
> > It should be just ignore_unused_variable, IMO. It doesn't ignore the
> > warning, it suppresses or avoids it.
> To be semantically correct it could be named use_variable() or
> use_variables(). Or just unused_variable() / unused_variables() meaning
> "hey! those are unused variables!". And if the name was changed we might as
> well consider putting it in the different namespace, e.g.
> The currently used name is for backward compatibility. We can change it or
> provide both and deprecate the old one.
> Or just leave currently defined one in the ConceptCheck but in this case
> this pull request has less sense than it could.
I'm in favor of renaming. ignore_unused_variable or
suppress_unused_variable_warning looks more appealing to me. use_variable
means "do something with it" to me, which is not the intention of the
Is this a documented feature of ConceptCheck? If not, backward compatibility
is not an issue.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk