Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

geos: add missing <cstdint> includes for gcc-13 #743

Merged
merged 1 commit into from Nov 29, 2022
Merged

geos: add missing <cstdint> includes for gcc-13 #743

merged 1 commit into from Nov 29, 2022

Conversation

trofi
Copy link
Contributor

@trofi trofi commented Nov 29, 2022

Without the change build on gcc-13 fails as:

geos/include/geos/geomgraph/TopologyLocation.h:143:52: error: 'uint32_t' has not been declared
  143 |     bool isEqualOnSide(const TopologyLocation& le, uint32_t locIndex) const
      |                                                    ^~~~~~~~

Without the change build on `gcc-13` fails as:

    geos/include/geos/geomgraph/TopologyLocation.h:143:52: error: 'uint32_t' has not been declared
      143 |     bool isEqualOnSide(const TopologyLocation& le, uint32_t locIndex) const
          |                                                    ^~~~~~~~
@dbaston dbaston merged commit 0e8d436 into libgeos:main Nov 29, 2022
@trofi trofi deleted the gcc-13-fix branch November 29, 2022 23:55
pramsey pushed a commit that referenced this pull request Jan 27, 2023
Without the change build on `gcc-13` fails as:

    geos/include/geos/geomgraph/TopologyLocation.h:143:52: error: 'uint32_t' has not been declared
      143 |     bool isEqualOnSide(const TopologyLocation& le, uint32_t locIndex) const
          |                                                    ^~~~~~~~
robe2 added a commit to robe2/geos that referenced this pull request Nov 12, 2023
robe2 added a commit that referenced this pull request Nov 12, 2023
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.

None yet

2 participants