You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've also created Pull Request with my propesed fix for this issue: #141
Issue Description:
Attempting to build the Maximilian library (or its examples) using standard cmake and make commands results in compilation errors. The errors seem to be related to integer comparison sign mismatches and an undefined type 'uint32_t'.
Steps to Reproduce:
clone the maximilian repository
navigate to any example, eg cd Maximilian/cpp/commandline/maximilian_examples/1.TestTone/
mkdir build
cd build
cmake ..
make
Observe the compilation errors (truncated):
[ 20%] Building CXX object CMakeFiles/maximilian.dir/main.cpp.o
In file included from /home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/main.cpp:1:
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h: In member function ‘double maxiRingBuf::reduce(size_t, reduceFunction, double)’:
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h:472:32: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
472 | for(int i=idx-N; i < idx; i++) {
| ~~^~~~~
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h:477:54: warning: comparison of integer expressions of different signedness: ‘int’ and ‘std::vector<double>::size_type’ {aka ‘long unsigned int’} [-Wsign-compare]
477 | for(int i=F64_ARRAY_SIZE(buf)-(N-idx); i < buf.size(); i++) {
| ~~^~~~~~~~~~~~
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h:481:28: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
481 | for(int i=0; i < idx; i++) {
| ~~^~~~~
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h: At global scope:
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h:1812:13: error: ‘uint32_t’ does not name a type
1812 | typedef uint32_t bitsig;
| ^~~~~~~~
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h:52:1: note: ‘uint32_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
51 | #include <numeric>
+++ |+#include <cstdint>
52 |
/home/lzieniew/Documents/Maximilian/cpp/commandline/maximilian_examples/1.TestTone/../../../../src/maximilian.h:1818:12: error: ‘bitsig’ does not name a type
1818 | static bitsig sig(bitsig v) { return v; }
| ^~~~~~
Expected Behavior:
The project should compile successfully without errors.
Actual Behavior:
Compilation fails with multiple errors.
I've also checked minimal build of Maximilian on docker, with images gcc12 and gcc13, and on gcc 13 I had the same error, while the gcc12 it compiled ok (in both I had to also install cmake and libasound2-dev) - so I guess the issue is caused be some changes in gcc 13
The text was updated successfully, but these errors were encountered:
I've also created Pull Request with my propesed fix for this issue: #141
Issue Description:
Attempting to build the Maximilian library (or its examples) using standard cmake and make commands results in compilation errors. The errors seem to be related to integer comparison sign mismatches and an undefined type 'uint32_t'.
Steps to Reproduce:
cd Maximilian/cpp/commandline/maximilian_examples/1.TestTone/
mkdir build cd build cmake .. make
Observe the compilation errors (truncated):
Expected Behavior:
The project should compile successfully without errors.
Actual Behavior:
Compilation fails with multiple errors.
System Information:
I've also checked minimal build of Maximilian on docker, with images gcc12 and gcc13, and on gcc 13 I had the same error, while the gcc12 it compiled ok (in both I had to also install cmake and libasound2-dev) - so I guess the issue is caused be some changes in gcc 13
The text was updated successfully, but these errors were encountered: