Skip to content

Commit f5312d8

Browse files
committed
Make rb_concurrent_set_funcs const
We should never modify rb_concurrent_set_funcs during runtime, so we can make it const.
1 parent 061d479 commit f5312d8

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

concurrent_set.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ struct concurrent_set {
2121
rb_atomic_t size;
2222
unsigned int capacity;
2323
unsigned int deleted_entries;
24-
struct rb_concurrent_set_funcs *funcs;
24+
const struct rb_concurrent_set_funcs *funcs;
2525
struct concurrent_set_entry *entries;
2626
};
2727

@@ -51,7 +51,7 @@ static const rb_data_type_t concurrent_set_type = {
5151
};
5252

5353
VALUE
54-
rb_concurrent_set_new(struct rb_concurrent_set_funcs *funcs, int capacity)
54+
rb_concurrent_set_new(const struct rb_concurrent_set_funcs *funcs, int capacity)
5555
{
5656
struct concurrent_set *set;
5757
VALUE obj = TypedData_Make_Struct(0, struct concurrent_set, &concurrent_set_type, set);

internal/concurrent_set.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ struct rb_concurrent_set_funcs {
1313
rb_concurrent_set_create_func create;
1414
};
1515

16-
VALUE rb_concurrent_set_new(struct rb_concurrent_set_funcs *funcs, int capacity);
16+
VALUE rb_concurrent_set_new(const struct rb_concurrent_set_funcs *funcs, int capacity);
1717
VALUE rb_concurrent_set_find_or_insert(VALUE *set_obj_ptr, VALUE key, void *data);
1818
VALUE rb_concurrent_set_delete_by_identity(VALUE set_obj, VALUE key);
1919
void rb_concurrent_set_foreach_with_replace(VALUE set_obj, int (*callback)(VALUE *key, void *data), void *data);

string.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -548,7 +548,7 @@ fstring_concurrent_set_create(VALUE str, void *data)
548548
return str;
549549
}
550550

551-
static struct rb_concurrent_set_funcs fstring_concurrent_set_funcs = {
551+
static const struct rb_concurrent_set_funcs fstring_concurrent_set_funcs = {
552552
.hash = fstring_concurrent_set_hash,
553553
.cmp = fstring_concurrent_set_cmp,
554554
.create = fstring_concurrent_set_create,

0 commit comments

Comments
 (0)