hpx/util/safe_lexical_cast.hpp is added.

[AntonBikineev]

Some unprotected uses of boost::lexical_cast are wrapped by safe_lexical_cast. Issue #799.
I tried to use safe_lexical_cast where string casts to number and lexical_cast is not surrounded by try-catch block. Note that some default values should be added.

[hkaiser]

Looks good to me

[hkaiser]

Some default values can be derived from the surrounding code, like for instance in this case:

std::string thread_pool_size = 
    ini.get_entry(key + ".io_pool_size", "2"); 
return hpx::util::safe_lexical_cast<std::size_t>(thread_pool_size, 2);

Could you go through the code one more time and change these uses of util::safe_lexical_cast?

[AntonBikineev]

Sure.

[sithhell]

The code uses C++11 uniform initialization. I'm not sure if it's supported
on all our supported compilers.

[hkaiser]

The code uses C++11 uniform initialization. I'm not sure if it's supported
on all our supported compilers.

Good point!

@AntonBikineev: with HPX we currently have to support older compilers not supporting all of C++11. Uniform initialization syntax is unfortunately one of those features not available to us. In your case that should be a simple fix, though, assuming the target type is default constructible.

[AntonBikineev]

I'm not sure, but if get_entry_as function does make sense it might be better to move it to more appropriate header.

[hkaiser]

I think it's fine to have it where you put it for now.