Add missing cstdint include for uint32_t

OPENDAP / libdap4

A new version of libdap that contains both DAP2 and DAP4 support

GNU Lesser General Public License v2.1

13 stars 14 forks source link

Add missing cstdint include for uint32_t #215

Closed opoplawski closed 1 year ago

opoplawski commented 1 year ago

Builds started to fail in fedora rawhide with gcc 13 with:

In file included from Constructor.cc:38:
crc.h:9:14: error: 'uint32_t' does not name a type
    9 | static const uint32_t kCrc32Table[256] = {
      |              ^~~~~~~~
crc.h:1:1: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
  +++ |+#include <cstdint>
    1 | /*

This fixes it.

jgallagher59701 commented 1 year ago

Merged via a separate PR. Since our builds depend on another build stored in S3, AWS auth issues prevent PRs from forks from working - we need to sort that out. But in the meantime, I move the PR to a branch in my copy of the repo and run a PR on that.

Thanks very much for this Orion!

James