Age	Commit message (Collapse)	Author	Lines
17 hours	Trim trailing slashes with git's strbuf helper	Bryce Kwon	-12/+4
17 hours	Build the full current url from the current url	Bryce Kwon	-8/+3
17 hours	Fold the va_list error page into its one caller	Bryce Kwon	-13/+4
17 hours	Ask head_is_named before carrying the branch	Bryce Kwon	-1/+1
17 hours	Let self_link's fallback be a plain reporevlink	Bryce Kwon	-8/+2
17 hours	Escape a submodule link's class like the others	Bryce Kwon	-6/+6
17 hours	Open site and repository links through one helper	Bryce Kwon	-24/+17
17 hours	Write each query argument through one helper	Bryce Kwon	-49/+30
17 hours	Return repolink's delimiter without copying it	Bryce Kwon	-1/+1
17 hours	Drop the query-string form of every link	Bryce Kwon	-86/+31
	cgit falls back to SCRIPT_NAME for virtual_root whenever cgitrc leaves it out, so the ?url= and ?r= forms only ever reached an error page raised while cgitrc was still being read. The page argument of cgit_add_hidden_formfields fed only the url field, so it is unused until its callers stop passing it.
17 hours	Stop leaking the host on an embedded page	Bryce Kwon	-3/+3
17 hours	Stop leaking the short hash of an object link	Bryce Kwon	-7/+4
17 hours	Trim a snapshot name without reading out of bounds	Bryce Kwon	-29/+12
	For a url of .git, or an empty one, repo_basename stepped its index to the byte before its buffer, read it, and would have cleared it had it held a slash. Git's strbuf trims take the same steps safely, and they lift the 1024-byte cap that failed the page for a longer url.
17 hours	Drop two includes nothing in their files uses	Bryce Kwon	-3/+0
17 hours	Give stdout back from a slot in one place	Bryce Kwon	-10/+11
17 hours	Use the parser's INT_MAX for every snapshot format	Bryce Kwon	-5/+1
	The snapshots parser already answers all with INT_MAX, so cgit.c needed no value of its own for the same mask.
17 hours	Hand back the fallback branch rather than a copy	Bryce Kwon	-13/+4
17 hours	Drop dead checks and a repeated filter cleanup	Bryce Kwon	-12/+7
	build_snapshot_setting always returns a buffer, both commit-sort values are tested on their own, authenticate_post ends the process, and the filters are cleaned up by the atexit handler once cmd_main returns.
17 hours	Cap the search with xstrndup alone	Bryce Kwon	-4/+1
17 hours	Return nothing from two calls nobody checks	Bryce Kwon	-9/+7
17 hours	Drop reencode's fixed target and unread result	Bryce Kwon	-12/+11
	Every call converted into the page encoding and dropped what came back, and a commit's encoding is always set by the time it runs.
17 hours	Rename prepare_repo_env to open_repo	Bryce Kwon	-2/+2
	It opens the repository and loads its notes, while cgit_prepare_repo_env exports the CGIT_REPO_ variables, and the two names were one prefix apart.
17 hours	Explain the notes setup in one comment	Bryce Kwon	-4/+3
17 hours	Leave the scan's summary to its header	Bryce Kwon	-7/+3
17 hours	Drop the scan's redundant stats and one-pass loop	Bryce Kwon	-12/+11
	The description and cgitrc readers already return when the file will not open, a repository's description is never NULL before it is read, and the owner lookup never went round a second time.
17 hours	Strip a scanned path's suffixes with git's helpers	Bryce Kwon	-10/+7
17 hours	Give the Lua-less filter init the Lua block's else	Bryce Kwon	-6/+4
17 hours	Allocate the filters zeroed with xcalloc	Bryce Kwon	-7/+2
17 hours	Drop the exec filter's close steps that do nothing	Bryce Kwon	-7/+1
	A filter is only closed after an open has forked it, so its pid is a child's, and every open writes its arguments into argv afresh.
17 hours	Format the cache listing's times where it prints	Bryce Kwon	-17/+6
17 hours	Close the cache's descriptors through one helper	Bryce Kwon	-33/+15
17 hours	Drop the cache's checks for arguments never NULL	Bryce Kwon	-14/+0
	The cache root always holds a default or a copy, the key comes from a strbuf, and every string hashed is the same or a macro expansion.
17 hours	Let scan_tree choose the project list itself	Bryce Kwon	-15/+8
	Three callers wrote out the same choice, and the header already says a scan visits only the listed paths when a project list is set.
17 hours	Drop the command line switches the query covers	Bryce Kwon	-11/+0
	--repo= only stored the name, so it never selected a repository, and --query= already reaches what the other four set, with the offset clamped the way a request's is.
17 hours	Pick a filter's backend with skip_prefix	Bryce Kwon	-32/+14
	A two entry table and a prefix measured by hand did what two prefix tests do. A drive letter and an unknown prefix are read as before.
17 hours	Write the cache's errors with fprintf like the rest	Bryce Kwon	-25/+10
17 hours	Drop the defaults the memset already sets	Bryce Kwon	-17/+0
	prepare_context zeroes the whole context first, and the repository list is a global, so neither needs its zeros written out again.
17 hours	Copy the encoding and subject with xmemdupz	Bryce Kwon	-18/+2
	substr was xmemdupz with a branch for an end before the start, and both callers find the end by searching forward from the start.
17 hours	Mark the static pages in the command table	Bryce Kwon	-38/+16
	The table already records what each page needs, and a second list of page names in cgit.c had to be kept in step with it by hand.
17 hours	Drop the thunks that only forward to a renderer	Bryce Kwon	-40/+10
17 hours	Set the virtual root from SCRIPT_NAME up front	Bryce Kwon	-7/+5
	The script name always holds a value, so the fallback after parsing fired whenever cgitrc left virtual-root unset. The root is now set with the other defaults, for cgitrc to replace. A die while cgitrc is read is the one change, and its page now links the root with the trailing slash every other page has. The environment's copy of the script name only fed this.
17 hours	Open the empty page's clone table on its first url	Bryce Kwon	-9/+12
	The page guessed from the two settings cgit_add_clone_urls reads, so a clone-url that expanded to nothing, or an empty one turning the global off, still drew a Clone heading over no rows. The summary already waits for a first url, and both now draw the heading the same way.
17 hours	Name the right cap in the ref listings' comment	Bryce Kwon	-1/+1
17 hours	Return nothing from the ref row printers	Bryce Kwon	-7/+4
17 hours	Drop ref date checks that cannot fire	Bryce Kwon	-3/+1
	A ref whose object git cannot parse never joins the list, and a commit's info is allocated even when its buffer cannot be read.
17 hours	Drop tag page checks that cannot fire	Bryce Kwon	-11/+7
	parse_object only hands back a tag it has parsed, and an ident is split into a name and an address together or not at all, so a tagger always has an address, if only <>.
17 hours	Write the feed's own url once	Bryce Kwon	-2/+2
17 hours	Let cgit_pageurl join a feed entry's commit id	Bryce Kwon	-10/+2
17 hours	Drop rev settings the feed's walk never reads	Bryce Kwon	-4/+0
17 hours	Strip a snapshot name's suffixes once	Bryce Kwon	-31/+21