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

Rectify the import paths of boost::function_output_iterator #4293

Merged
merged 2 commits into from
Mar 15, 2023

Conversation

ckeshava
Copy link
Collaborator

@ckeshava ckeshava commented Sep 5, 2022

Rectify the import paths of boost/iterator as suggested by compiler warnings

High Level Overview of Change

MSVC 19.x reports a warning about import paths in boost for function_output_iterator class. This commit aims to eliminate that warning by updating the import paths.

  • [ X ] Refactor (non-breaking change that only restructures code)

Copy link
Contributor

@HowardHinnant HowardHinnant left a comment

Choose a reason for hiding this comment

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

Needs rebasing to develop.

@intelliot intelliot changed the base branch from develop to next February 23, 2023 05:42
@intelliot intelliot changed the base branch from next to develop February 23, 2023 05:42
@intelliot intelliot requested a review from drlongle February 23, 2023 05:43
@intelliot intelliot added this to the 1.10.1 milestone Feb 23, 2023
@drlongle
Copy link
Contributor

LGTM

@intelliot
Copy link
Collaborator

@ckeshava: At your convenience, please comment here to confirm that, from your perspective, this PR is ready to merge.

@ckeshava
Copy link
Collaborator Author

ckeshava commented Mar 8, 2023

@intelliot Hello, it looks okay to me.

I don't have the toolchain to unit-test this change with the most recent version of develop. If the unit tests are working correctly, then it can be merged. I'm hoping there haven't been any changes to boost or rippled's compatibility.

@intelliot intelliot added the Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required. label Mar 8, 2023
@intelliot intelliot merged commit 9309b57 into XRPLF:develop Mar 15, 2023
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
…ctionality

* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
ximinez added a commit to ximinez/rippled that referenced this pull request Mar 15, 2023
…tpage

* upstream/develop:
  Rectify the import paths of boost/iterator: (XRPLF#4293)
  Allow port numbers be be specified with a colon: (XRPLF#4328)
  Use <=> operator for base_uint, Issue, and Book: (XRPLF#4411)
  Reporting Mode: Do not attempt to acquire missing data from peer network (XRPLF#4458)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants