Skip to content

Add missing headers for C++23 - #2029

Merged
intel-mediadev merged 1 commit into
intel:masterfrom
rosetta-jpn:c++23
Sep 11, 2026
Merged

intel-mediadev merged 1 commit into
intel:masterfrom
rosetta-jpn:c++23

Conversation

@rosetta-jpn

Copy link
Copy Markdown
Contributor

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
Contributor Author

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

@XinfengZhang XinfengZhang added the verifying PR: fix ready and verifying with build/test label Sep 10, 2026
@intel-mediadev
intel-mediadev merged commit 120c4dd into intel:master Sep 11, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

verifying PR: fix ready and verifying with build/test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants