Thread (69 messages) 69 messages, 3 authors, 13h ago

Re: [PATCH v2 2/5] setup: extract path_allowlist_apply()

From: Junio C Hamano <hidden>
Date: 2026-09-08 17:49:18

Christian Couder [off-list ref] writes:
path_allowlist_apply(key, value, data->path, &data->is_safe);

But OK, I have changed both `int is_match` and `int is_safe` to `bool
matches` and `bool safe` in the v3 I just sent.
I hate to say this but I think is_safe was perfectly good.  is_match
was not quite grammatrical (it is either "matches" ir "is_a_match"),
but "is_safe" is perfectly fine.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help