Github/littlefs
1
Fork 0
Code Actions Activity

rbyd: Renamed LFS_ASSERTRBYDBALANCE -> LFS_DEBUGRBYDBALANCE

Browse Source 
This name helps avoid the mistake the that LFS_ASSERTRBYDBALANCE is
cheap/free like other asserts, which is very much not true.
LFS_DEBUGRBYDBALANCE is expensive and should only be used for testing.
This commit is contained in:
Christopher Haster
2025-01-27 23:33:45 -06:00
parent dbf0b5ebb1
commit 558911693b
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lfs.c
+1 -1
View File
@@ -2939,7 +2939,7 @@ static int lfsr_rbyd_fetch_(lfs_t *lfs,
// asserting rbyd balance? check that all branches in the rbyd have // asserting rbyd balance? check that all branches in the rbyd have
// the same height // the same height
#ifdef LFS_ASSERTRBYDBALANCE #ifdef LFS_DEBUGRBYDBALANCE
lfsr_srid_t rid = -1; lfsr_srid_t rid = -1;
lfsr_tag_t tag = 0; lfsr_tag_t tag = 0;
lfs_ssize_t height = -1; lfs_ssize_t height = -1;
lfs_util.h
+3 -3
View File
@@ -168,10 +168,10 @@ extern "C"
#define LFS_IFDEF_GC(a, b) (b) #define LFS_IFDEF_GC(a, b) (b)
#endif #endif
#ifdef LFS_ASSERTRBYDBALANCE #ifdef LFS_DEBUGRBYDBALANCE
#define LFS_IFDEF_ASSERTRBYDBALANCE(a, b) (a) #define LFS_IFDEF_DEBUGRBYDBALANCE(a, b) (a)
#else #else
#define LFS_IFDEF_ASSERTRBYDBALANCE(a, b) (b) #define LFS_IFDEF_DEBUGRBYDBALANCE(a, b) (b)
#endif #endif