Looks good from a functionality perspective. Some style nits:
8 + if (activate) {
opening brace on its own line
10 + BOOST_THROW_EXCEPTION(
11 + boost::enable_error_info(
12 + std::runtime_error("Failed to stop being a process group"))
13 + << boost::errinfo_errno(errno));
17 + BOOST_THROW_EXCEPTION(
18 + boost::enable_error_info(
19 + std::runtime_error("Failed to become session leader"))
20 + << boost::errinfo_errno(errno));
Multi-line statements in 'if' clauses should be placed in curly braces.
Looks good from a functionality perspective. Some style nits:
8 + if (activate) {
opening brace on its own line
10 + BOOST_THROW_ EXCEPTION( enable_ error_info( error(" Failed to stop being a process group")) errinfo_ errno(errno) );
11 + boost::
12 + std::runtime_
13 + << boost::
17 + BOOST_THROW_ EXCEPTION( enable_ error_info( error(" Failed to become session leader")) errinfo_ errno(errno) );
18 + boost::
19 + std::runtime_
20 + << boost::
Multi-line statements in 'if' clauses should be placed in curly braces.