Skip to content

Fix util namespaces to use the project with C++17 applications#1232

Closed
sergeymarkow wants to merge 1 commit into
libcpr:masterfrom
sergeymarkow:master
Closed

Fix util namespaces to use the project with C++17 applications#1232
sergeymarkow wants to merge 1 commit into
libcpr:masterfrom
sergeymarkow:master

Conversation

@sergeymarkow

@sergeymarkow sergeymarkow commented Jul 3, 2025

Copy link
Copy Markdown

It is ok to use library which is built with newer C++ standart but it will be great to have including headers to keep C++17 standart.

Unfortunatly I need to use Qt5.15 which still use C++17. Moreover I cannot rebuild Qt5.15 with other C++ params I can use already built project ((. I think in some other cases such change can be required.

@COM8 COM8 self-requested a review July 5, 2025 05:49
@COM8

COM8 commented Jul 5, 2025

Copy link
Copy Markdown
Member

@sergeymarkow thanks for contributing!
But I always thought this was c++17 and also the standard says so: https://en.cppreference.com/w/cpp/language/namespace.html

Is this a special case where QT5 does not support this?

@COM8

COM8 commented Aug 10, 2025

Copy link
Copy Markdown
Member

Closing for no response. Feel free to reopen in case this is still a problem.

@COM8 COM8 closed this Aug 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants