Skip to content

Fix build with GCC 13 (add missing <cstdint> include)#1089

Closed
thesamesam wants to merge 1 commit into
facebook:mainfrom
thesamesam:gcc-13
Closed

Fix build with GCC 13 (add missing <cstdint> include)#1089
thesamesam wants to merge 1 commit into
facebook:mainfrom
thesamesam:gcc-13

Conversation

@thesamesam
Copy link
Copy Markdown
Contributor

GCC 13 (as usual for new compiler releases) shuffles around some internal includes and so is no longer transitively included.

Explicitly include for uint32_t.

Signed-off-by: Sam James sam@gentoo.org

GCC 13 (as usual for new compiler releases) shuffles around some
internal includes and so <cstdint> is no longer transitively included.

Explicitly include <cstdint> for uint32_t.

Signed-off-by: Sam James <sam@gentoo.org>
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@yfeldblum has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@yfeldblum merged this pull request in 7220385.

facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request Jan 23, 2023
Summary:
GCC 13 (as usual for new compiler releases) shuffles around some internal includes and so <cstdint> is no longer transitively included.

Explicitly include <cstdint> for uint32_t.

Signed-off-by: Sam James <sam@gentoo.org>

X-link: facebook/watchman#1089

Reviewed By: genevievehelsel

Differential Revision: D42670019

Pulled By: yfeldblum

fbshipit-source-id: 9af4d6ee546e72fe0ab2f1d93894db46a2f4d099
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