Thread (1 message) 1 message, 1 author, 2021-10-21

Re: [PATCH v2 1/3] refs.c: make "repo_default_branch_name" static, remove xstrfmt()

From: Junio C Hamano <hidden>
Date: 2021-10-21 23:26:31

Ævar Arnfjörð Bjarmason  [off-list ref] writes:
In addition the xstrfmt() to get the "full_ref" in the same commit
isn't needed, we can use the "REFNAME_ALLOW_ONELEVEL" flag to
check_refname_format() instead.
Reading the code of check_refname_format(), I do not think one-level
is the only thing that the prefixing of refs/heads/ is defeating,
and more importantly, I'd expect that this will block later changes
like enforcing "HEAD might be OK in onelevel because we want to keep
.git/HEAD working, but we do not like refs/heads/HEAD" at this level
to enhance usability from happening.

Making the function file-local static is a good thing to do, though.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help