Always define the is_power_of_two function (#36)

st0012 · web-flow · commit 91dfbbda3cc4 · 2025-04-02T15:30:36.000+01:00
It was only used in `assert` macros, so it only needs to be defined in
debug mode. But since `rbs_assert` is a function, `is_power_of_two`
always needs to be defined.

Without this change, Sorbet fails to build with `c-api`.
diff --git a/src/util/rbs_constant_pool.c b/src/util/rbs_constant_pool.c
@@ -37,12 +37,9 @@ next_power_of_two(uint32_t v) {
     return v;
 }
 
-#ifndef NDEBUG
-static bool
-is_power_of_two(uint32_t size) {
+static bool is_power_of_two(uint32_t size) {
     return (size & (size - 1)) == 0;
 }
-#endif
 
 /**
  * Resize a constant pool to a given capacity.

Original file line number	Diff line number	Diff line change
`@@ -37,12 +37,9 @@ next_power_of_two(uint32_t v) {`
`37`	`37`	`return v;`
`38`	`38`	`}`
`39`	`39`
`40`		`-#ifndef NDEBUG`
`41`		`-static bool`
`42`		`-is_power_of_two(uint32_t size) {`
	`40`	`+static bool is_power_of_two(uint32_t size) {`
`43`	`41`	`return (size & (size - 1)) == 0;`
`44`	`42`	`}`
`45`		`-#endif`
`46`	`43`
`47`	`44`	`/**`
`48`	`45`	`* Resize a constant pool to a given capacity.`