Skip to content

AudioLocation.h: add missing <cstdint> include - #1086

Open
Chzrz89 wants to merge 1 commit into
OpenShot:developfrom
Chzrz89:fix/audiolocation-cstdint
Open

AudioLocation.h: add missing <cstdint> include#1086
Chzrz89 wants to merge 1 commit into
OpenShot:developfrom
Chzrz89:fix/audiolocation-cstdint

Conversation

@Chzrz89

@Chzrz89 Chzrz89 commented Aug 26, 2026

Copy link
Copy Markdown

Summary

One-line fix: add #include <cstdint> to src/AudioLocation.h.

The struct uses int64_t but relied on it arriving via transitive includes. GCC 16 no longer provides int64_t transitively, so the struct definition fails to parse:

H:/libopenshot/src/AudioLocation.h:26:17: error: 'int64_t' does not name a type
   26 |                 int64_t frame;
      |                 ^~~~~~~

which cascades into misleading struct openshot::AudioLocation has no member named frame errors at every use site in Clip.cpp.

Making the header self-contained fixes the build with no behavior change.

Testing

  • Full build passes on Windows/MSYS2 (MinGW-w64 GCC 16.2.0)
  • Header-only change; no effect on FFmpeg/OpenCV code paths or older compilers

The struct uses int64_t but relied on transitive includes for it.
GCC 16 no longer provides int64_t transitively, causing the struct
definition to fail parsing ('int64_t' does not name a type) and
cascading 'has no member frame' errors in Clip.cpp.

Explicitly include <cstdint> so the header is self-contained.
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