Skip to content

Add missing headers for C++23 - #2029

Open
rosetta-jpn wants to merge 1 commit into
intel:masterfrom
rosetta-jpn:c++23
Open

Add missing headers for C++23#2029
rosetta-jpn wants to merge 1 commit into
intel:masterfrom
rosetta-jpn:c++23

Conversation

@rosetta-jpn

Copy link
Copy Markdown

Add headers to resolve C++23 build errors because C++23 removed transitive includes so you have to add the headers directly.

Errors:
vendor/intel/desktop/media-driver/cmrtlib/agnostic/share/cm_printf_host.cpp:226:43: error: use of undeclared identifier 'atoi'
226 | m_currToken.tokenString =
atoi(m_currToken.tokenString.c_str());

Add headers to resolve C++23 build errors because C++23 removed
transitive includes so you have to add the headers directly.

Errors:
vendor/intel/desktop/media-driver/cmrtlib/agnostic/share/cm_printf_host.cpp:226:43:
error: use of undeclared identifier 'atoi'
  226 |                 m_currToken.tokenString =
atoi(m_currToken.tokenString.c_str());

Signed-off-by: Hirokazu Honda <hiroh@google.com>
@XinfengZhang

Copy link
Copy Markdown
Contributor

same as #2027
I prefer to put cm header as a group and standard header as another group. as this patch.

@XinfengZhang

Copy link
Copy Markdown
Contributor

/build

@rosetta-jpn

Copy link
Copy Markdown
Author

Hi @XinfengZhang, is any modification necessary to merge this patch?

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.

3 participants