Skip to content

Add algorithm header to ProviderConfig.cpp - #635

Merged
vt-tv merged 2 commits into
dmf-mxl:mainfrom
saycmily:patch-1
Jul 29, 2026
Merged

Add algorithm header to ProviderConfig.cpp#635
vt-tv merged 2 commits into
dmf-mxl:mainfrom
saycmily:patch-1

Conversation

@saycmily

Copy link
Copy Markdown
Contributor

no algorithm header will compile fail, std::ranges::find

Details

Backport requirements

no algorithm header will compile fail,  std::ranges::find

Signed-off-by: 曹勇 <1786386686@qq.com>

@KimonHoffmann KimonHoffmann left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me, thank you!

@jonasohland jonasohland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. @saycmily can you tell me what system and compiler you are seeing this problem on?

@saycmily

saycmily commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

LGTM. @saycmily can you tell me what system and compiler you are seeing this problem on?

ubuntu2204 x86_64, g++12.3, I use cmake preset Linux-GCC-Release

@KimonHoffmann

KimonHoffmann commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Please address the reported clang-format errors. <algorithm> needs to be ordered before the inclusion of "Exception.hpp".

Signed-off-by: 曹勇 <1786386686@qq.com>
@vt-tv vt-tv added the backport/v1.1 This PR should be back ported to the release branch of version 1.1. label Jul 29, 2026
@vt-tv
vt-tv merged commit fc8611c into dmf-mxl:main Jul 29, 2026
11 checks passed
@backport-mxl-pull-requst

Copy link
Copy Markdown

Successfully created backport PR for release/v1.1:

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

Labels

backport/v1.1 This PR should be back ported to the release branch of version 1.1.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants