Skip to content

include,src: add missing <cstdint> includes#31

Open
orlitzky wants to merge 1 commit into
keeganryan:mainfrom
orlitzky:missing-cstdint
Open

include,src: add missing <cstdint> includes#31
orlitzky wants to merge 1 commit into
keeganryan:mainfrom
orlitzky:missing-cstdint

Conversation

@orlitzky

Copy link
Copy Markdown

Without this, the build fails on systems with the musl C library, though GCC is nice enough to tell you what to do:

.../include/flatter/data/matrix/matrix.h:7:1: note: 'int64_t' is
defined in header '<cstdint>'; this is probably fixable by adding
'#include <cstdint>'
    6 | #include "matrix_data.h"
  +++ |+#include <cstdint>
    7 |

(Any file using the int64_t type should #include <cstdint>)

Without this, the build fails on systems with the musl C library,
though GCC is nice enough to tell you what to do:

  .../include/flatter/data/matrix/matrix.h:7:1: note: 'int64_t' is
  defined in header '<cstdint>'; this is probably fixable by adding
  '#include <cstdint>'
      6 | #include "matrix_data.h"
    +++ |+#include <cstdint>
      7 |

(Any file using the int64_t type should include <cstdint>)
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.

1 participant