[THRIFT-4678] add noexcept cpp generator option

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.11.0
• Fix Version/s: 0.13.0
• Component/s: C++ - Compiler, C++ - Library
• Patch Info: Patch Available
• Flags: Patch
• External issue URL: https://github.com/apache/thrift/pull/1599
• Assignee: James E. King III
• Reporter: yuanyuan chen
• Created: 07/Dec/18 02:41
• Updated: 16/Oct/19 22:27
• Resolved: 07/Jan/19 13:12

Description

The C++11 standard has deprecated the usage of throw() to express exceptions,so to avoid warnings from the compiler,I think this option is useful.

I have a pull request in github,this issue is created to track it.

Some questions remain:

1.Should we change the runtime c++ library to use BOOST_NOEXCEPT_OR_NOTHROW?

2.Should we add an control option to enable all c++11 options like moveable_types .etc?

3.Should we begin to support C+17 features? I think std::optional should be used to implement optional keyword,but this is clearly an API breaking change,so we need an c+17 control option.

Issue Links

• is related to: THRIFT-4720 Drop support for C++03/C++98 and begin refactoring
• relates to: THRIFT-4441 C++: support building lib without Boost
• links to: GitHub Pull Request #1599