littlefs-project / littlefs

A little fail-safe filesystem designed for microcontrollers
BSD 3-Clause "New" or "Revised" License
5.13k stars 791 forks source link

Warning: lfs.c #861

Open FiddlingBits opened 1 year ago

FiddlingBits commented 1 year ago

See: https://github.com/littlefs-project/littlefs/blob/611c9b20db2b99faee261daa7cc9bbe175d3eaca/lfs.c#L1631 Statement: uint8_t eperturb = -1; Warning: #69-D integer conversion resulted in a change of sign Environment: Code Composer Studio; MSP430 Possible Resolution: uint8_t eperturb = (uint8_t)-1; uint8_t eperturb = (uint8_t)~0;

geky commented 1 year ago

Hi @FiddlingBits, thanks for creating an issue. Put up your proposed fix here: https://github.com/littlefs-project/littlefs/pull/863, will merge on the next patch release.

DeflateAwning commented 3 months ago

Probably safe to close