averello/git-mirror
1
0
Fork 0
mirror of https://github.com/git/git.git synced 2026-03-01 18:24:00 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'jk/ref-filter-lrstrip-optim' into next

Browse Source 
Code clean-up.

* jk/ref-filter-lrstrip-optim:
  ref-filter: avoid strrchr() in rstrip_ref_components()
  ref-filter: simplify rstrip_ref_components() memory handling
  ref-filter: simplify lstrip_ref_components() memory handling
  ref-filter: factor out refname component counting
This commit is contained in:
Junio C Hamano
2026-02-19 14:29:06 -08:00
parent 9d5bc4efc5 fe732a8b9f
commit 6efc8a2023
1 changed files with 17 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

54
ref-filter.c
View File

@@ -2173,12 +2173,8 @@ static inline char *copy_advance(char *dst, const char *src)
return dst;
}
static const char *lstrip_ref_components(const char *refname, int len)
static int normalize_component_count(const char *refname, int len)
{
long remaining = len;
const char *start = xstrdup(refname);
const char *to_free = start;
if (len < 0) {
int i;
const char *p = refname;
@@ -2192,13 +2188,18 @@ static const char *lstrip_ref_components(const char *refname, int len)
* because we count the number of '/', but the number
* of components is one more than the no of '/').
*/
remaining = i + len + 1;
len = i + len + 1;
}
return len;
}
static const char *lstrip_ref_components(const char *refname, int len)
{
int remaining = normalize_component_count(refname, len);
while (remaining > 0) {
switch (*start++) {
switch (*refname++) {
case '\0':
free((char *)to_free);
return xstrdup("");
case '/':
remaining--;
@@ -2206,42 +2207,21 @@ static const char *lstrip_ref_components(const char *refname, int len)
}
}
start = xstrdup(start);
free((char *)to_free);
return start;
return xstrdup(refname);
}
static const char *rstrip_ref_components(const char *refname, int len)
{
long remaining = len;
const char *start = xstrdup(refname);
const char *to_free = start;
int remaining = normalize_component_count(refname, len);
const char *end = refname + strlen(refname);
if (len < 0) {
int i;
const char *p = refname;
/* Find total no of '/' separated path-components */
for (i = 0; p[i]; p[i] == '/' ? i++ : *p++)
;
/*
* The number of components we need to strip is now
* the total minus the components to be left (Plus one
* because we count the number of '/', but the number
* of components is one more than the no of '/').
*/
remaining = i + len + 1;
}
while (remaining-- > 0) {
char *p = strrchr(start, '/');
if (!p) {
free((char *)to_free);
while (remaining > 0) {
if (end == refname)
return xstrdup("");
} else
p[0] = '\0';
if (*--end == '/')
remaining--;
}
return start;
return xmemdupz(refname, end - refname);
}
static const char *show_ref(struct refname_atom *atom, const char *refname)