Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced boost facilities with C++ 11 std wherever available #425

Merged
merged 7 commits into from Jan 9, 2019

Conversation

thiru-mg
Copy link
Contributor

@thiru-mg thiru-mg commented Jan 7, 2019

The following have been moved:

  1. Boost smart pointers to std smart pointers
  2. boost::array to std::array
  3. boost::ptr_containers to std::vector<std::unique_ptr
  4. BOOST_STATIC_ASSERT to static_assert
  5. boost's type_traits to std type_traits

However, the following are not moved:

  1. We still use boost::share_ptr in tests because boost test does not accept std::share_ptr yet.
  2. Boost's noncopyable is more convenient than =delete copy constructor. So not moving to std there.

@thiru-mg thiru-mg self-assigned this Jan 7, 2019
@thiru-mg thiru-mg merged commit 7a09848 into apache:master Jan 9, 2019
@thiru-mg thiru-mg deleted the AVRO-2295 branch January 9, 2019 02:46
@iemejia iemejia added the C++ Pull Requests for C++ binding label Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ Pull Requests for C++ binding
Projects
None yet
3 participants