Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

32bit warnings #663

Open
Azq2 opened this issue Jan 5, 2025 · 0 comments
Open

32bit warnings #663

Azq2 opened this issue Jan 5, 2025 · 0 comments

Comments

@Azq2
Copy link

Azq2 commented Jan 5, 2025

This should use uint64_t or simply exclude this bit shifting for 32bit environments.

libhv/http/websocket_parser.c: In function 'websocket_build_frame':
libhv/http/websocket_parser.c:228:38: warning: right shift count >= width of type [-Wshift-count-overflow]
  228 |         frame[2] = (char) ((data_len >> 56) & 0xFF);
      |                                      ^~
libhv/http/websocket_parser.c:229:38: warning: right shift count >= width of type [-Wshift-count-overflow]
  229 |         frame[3] = (char) ((data_len >> 48) & 0xFF);
      |                                      ^~
libhv/http/websocket_parser.c:230:38: warning: right shift count >= width of type [-Wshift-count-overflow]
  230 |         frame[4] = (char) ((data_len >> 40) & 0xFF);
      |                                      ^~
libhv/http/websocket_parser.c:231:38: warning: right shift count >= width of type [-Wshift-count-overflow]
  231 |         frame[5] = (char) ((data_len >> 32) & 0xFF);
libhv/base/hbase.c:417:35: warning: left shift count >= width of type [-Wshift-count-overflow]
  417 |             case 'T': case 't': n <<= 40; break;
      |              

$ arm-linux-gnueabihf-gcc -v
Using built-in specs.
COLLECT_GCC=arm-linux-gnueabihf-gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/arm-linux-gnueabihf/14.2.0/lto-wrapper
Target: arm-linux-gnueabihf
Configured with: ../gcc-14.2.0/configure --target=arm-linux-gnueabihf --host=x86_64-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/usr --program-prefix=arm-linux-gnueabihf- --libdir=/usr/lib --libexecdir=/usr/lib --with-local-prefix=/usr/arm-linux-gnueabihf --with-sysroot=/usr/arm-linux-gnueabihf --with-build-sysroot=/usr/arm-linux-gnueabihf --with-native-system-header-dir=/include --with-as=/usr/bin/arm-linux-gnueabihf-as --with-ld=/usr/bin/arm-linux-gnueabihf-ld --with-linker-hash-style=gnu --with-system-zlib --with-arch=armv6 --with-float=hard --with-fpu=vfp --enable-languages=c,c++,lto,m2 --enable-__cxa_atexit --enable-cet=auto --enable-checking=release --enable-clocale=gnu --enable-default-pie --enable-default-ssp --enable-gnu-indirect-function --enable-gnu-unique-object --enable-libstdcxx-backtrace --enable-link-serialization=1 --enable-linker-build-id --enable-lto --enable-plugin --enable-shared --enable-threads=posix --enable-libquadmath --enable-libvtv --disable-nls --disable-install-libiberty --disable-libssp --disable-libstdcxx-pch --disable-multilib --disable-werror
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 14.2.0 (GCC) 
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant