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

LinterConfig.h: add missing stdint.h include #1010

Merged
merged 1 commit into from
Aug 14, 2023

Conversation

xgqt
Copy link
Contributor

@xgqt xgqt commented Aug 12, 2023

Without stdint.h following is raised while trying to build luau:

In file included from /home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/Config.h:4,
                 from /home/xy/source/public/github.com/Roblox/luau/Analysis/src/Config.cpp:2:
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:60:12: error: ‘uint64_t’ does not name a type
   60 |     static uint64_t parseMask(const std::vector<HotComment>& hotcomments);
      |            ^~~~~~~~
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:8:1: note: ‘uint64_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
    7 | #include <vector>
  +++ |+#include <cstdint>
    8 | 
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:65:5: error: ‘uint64_t’ does not name a type
   65 |     uint64_t warningMask = 0;
      |     ^~~~~~~~
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:65:5: note: ‘uint64_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h: In member function ‘void Luau::LintOptions::enableWarning(Luau::LintWarning::Code)’:
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:69:9: error: ‘warningMask’ was not declared in this scope
   69 |         warningMask |= 1ull << code;
      |         ^~~~~~~~~~~
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h: In member function ‘void Luau::LintOptions::disableWarning(Luau::LintWarning::Code)’:
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:73:9: error: ‘warningMask’ was not declared in this scope
   73 |         warningMask &= ~(1ull << code);
      |         ^~~~~~~~~~~
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h: In member function ‘bool Luau::LintOptions::isEnabled(Luau::LintWarning::Code) const’:
/home/xy/source/public/github.com/Roblox/luau/Analysis/include/Luau/LinterConfig.h:78:22: error: ‘warningMask’ was not declared in this scope
   78 |         return 0 != (warningMask & (1ull << code));
      |                      ^~~~~~~~~~~
[ 23%] Building CXX object CMakeFiles/Luau.Analysis.dir/Analysis/src/ConstraintGraphBuilder.cpp.o
make[2]: *** [CMakeFiles/Luau.Analysis.dir/build.make:174: CMakeFiles/Luau.Analysis.dir/Analysis/src/

Signed-off-by: Maciej Barć <xgqt@gentoo.org>
@xgqt
Copy link
Contributor Author

xgqt commented Aug 13, 2023

CC @zeux because I saw you take care of #925

@vegorov-rbx vegorov-rbx merged commit a2a4710 into luau-lang:master Aug 14, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants