boost_program_options.1.55.0.16\lib\native\src\value_semantic.cpp(271): warning C4127: conditional expression is constant

[nn1436401@…]

Instead of while(1) use for(;;)

void error_with_option_name::replace_token(const string& from, const string& to) const
    {
        while (1)
        {

[Vladimir Prus]

For the obvious fix like this, could you make a pull request on github? I think it has "Edit" button that would allow to create such request without any local work at all.

And, while we're at it, MSVC is not acting smart here ;-)

[nn1436401@…]

I see fix here: github.com/boostorg/program_options/blob/develop/src/value_semantic.cpp#L271

What is the reason for not using 'for(;;)' and then there will be no need for #ifdef ?

[Vladimir Prus]

#ifdefing while(1) does not seem reasonable, indeed.

[Vladimir Prus]

Fixed in ​https://github.com/boostorg/program_options/commit/37804b54d4ecbeaea770e4f4fce46cc371083c81