intel / ledmon

Enclosure LED Utilities
GNU General Public License v2.0
73 stars 47 forks source link

Correct type warnings found while using clang #164

Closed tasleson closed 1 year ago

tasleson commented 1 year ago

Corrects a number of type warnings that were found when I build the project using clang instead of gcc.

$ clang --version
clang version 16.0.6 (Fedora 16.0.6-2.fc38)
Target: x86_64-redhat-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

$ ./autogen.sh && CC=clang ./configure --enable-test --enable-library  && make -j12
tasleson commented 1 year ago

One small nit

Great catch, thank you. It was mistakenly done when I was doing a search & replace.

mtkaczyk commented 1 year ago

@tasleson, I tried to resolve conflict manually but it seems to be broken now. Please push force..

Thanks!

tasleson commented 1 year ago

Consolidates changes that were in https://github.com/intel/ledmon/pull/168