History

selftests/harness: Constify fixture variants

FIXTURE_VARIANT_ADD() types are passed as const pointers to FIXTURE_TEARDOWN(). Make that explicit by constifying the variants declarations.

Mickaël Salaün committed 2 years ago

Commit cc80aa91 parent 3656bc21 file · +2 −2Browse files

Unified · Split

tools/testing/selftests/kselftest_harness.h+2−2
@@ -338,7 +338,7 @@ static inline pid_t clone3_vfork(void) · 337 unmodified lines
338 * variant.338 * variant.
339 */339 */
340#define FIXTURE_VARIANT_ADD(fixture_name, variant_name) \340#define FIXTURE_VARIANT_ADD(fixture_name, variant_name) \
341 extern FIXTURE_VARIANT(fixture_name) \341 extern const FIXTURE_VARIANT(fixture_name) \
342 _##fixture_name##_##variant_name##_variant; \342 _##fixture_name##_##variant_name##_variant; \
343 static struct __fixture_variant_metadata \343 static struct __fixture_variant_metadata \
344 _##fixture_name##_##variant_name##_object = \344 _##fixture_name##_##variant_name##_object = \
@@ -350,7 +350,7 @@ static inline pid_t clone3_vfork(void) · 5 unmodified lines
350 __register_fixture_variant(&_##fixture_name##_fixture_object, \350 __register_fixture_variant(&_##fixture_name##_fixture_object, \
351 &_##fixture_name##_##variant_name##_object); \351 &_##fixture_name##_##variant_name##_object); \
352 } \352 } \
353 FIXTURE_VARIANT(fixture_name) \353 const FIXTURE_VARIANT(fixture_name) \
354 _##fixture_name##_##variant_name##_variant =354 _##fixture_name##_##variant_name##_variant =
355 355
356/**356/**