Fix legacy __rt_ffs_tiny()

pegasusplus · web-flow · commit 0e9c6c830f7c · 2024-12-01T00:48:47.000-08:00
Some compiler complains value &amp; (value - 1) ^ value
better (value &amp; (value - 1)) ^ value
diff --git a/src/kservice.c b/src/kservice.c
@@ -1092,7 +1092,7 @@ const rt_uint8_t __lowest_bit_bitmap[] =
  */
 int __rt_ffs(int value)
 {
-    return __lowest_bit_bitmap[(rt_uint32_t)(value & (value - 1) ^ value) % 37];
+    return __lowest_bit_bitmap[(rt_uint32_t)((value & (value - 1)) ^ value) % 37];
 }
 #else
 const rt_uint8_t __lowest_bit_bitmap[] =

Original file line number	Diff line number	Diff line change
`@@ -1092,7 +1092,7 @@ const rt_uint8_t __lowest_bit_bitmap[] =`
`1092`	`1092`	`*/`
`1093`	`1093`	`int __rt_ffs(int value)`
`1094`	`1094`	`{`
`1095`		`- return __lowest_bit_bitmap[(rt_uint32_t)(value & (value - 1) ^ value) % 37];`
	`1095`	`+ return __lowest_bit_bitmap[(rt_uint32_t)((value & (value - 1)) ^ value) % 37];`
`1096`	`1096`	`}`
`1097`	`1097`	`#else`
`1098`	`1098`	`const rt_uint8_t __lowest_bit_bitmap[] =`