Skip to content

Mostly remove pre-CXX11 workarounds.#82

Closed
gogagum wants to merge 17 commits intoboostorg:developfrom
gogagum:develop
Closed

Mostly remove pre-CXX11 workarounds.#82
gogagum wants to merge 17 commits intoboostorg:developfrom
gogagum:develop

Conversation

@gogagum
Copy link
Contributor

@gogagum gogagum commented Jun 10, 2024

С++03 support was deprecated in 1.85 and now can be removed. This PR clears many of workarounds, which are no longer needed now.

  • Remove unused workaround macros (many of).
  • Remove BOOST_STATIC_ASSERT usages.
  • Minimize Boost::type_traits dependency (in favour of STL's type_traits).

@gogagum gogagum force-pushed the develop branch 4 times, most recently from 65f155a to 9e5a65f Compare June 12, 2024 15:24
@Lastique
Copy link
Member

Also, as a general comment, please convert this to a draft PR until you consider everything done and ready for review.

@gogagum gogagum marked this pull request as draft June 24, 2024 08:03
@gogagum gogagum force-pushed the develop branch 3 times, most recently from fa28bec to e49506b Compare October 10, 2024 22:46
@gogagum gogagum marked this pull request as ready for review October 10, 2024 22:58
@Lastique Lastique marked this pull request as draft October 11, 2024 16:03
@gogagum gogagum marked this pull request as ready for review December 21, 2024 23:33
@gogagum gogagum requested a review from Lastique January 18, 2025 11:36
Copy link
Member

@Lastique Lastique left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, you're almost there. Only a few minor comments left.

@gogagum gogagum force-pushed the develop branch 2 times, most recently from 141001a to 12284f6 Compare January 19, 2025 16:23
@Lastique Lastique closed this in f9effff Jan 26, 2025
@Lastique
Copy link
Member

Merged with some amendments. Thank you for your work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants