mvhutz/lambda
1
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use loop variable instead of global ticker in GenerateFreshName #40

Merged
mvhutz merged 1 commits from fix/fresh-name-generation into main 2026-01-18 20:58:24 +00:00
Conversation 0 Commits 1 Files Changed 1 +1 -3
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/lambda/generate_name.go
View File

@@ -6,13 +6,11 @@ import (
"git.maximhutz.com/max/lambda/pkg/set" "git.maximhutz.com/max/lambda/pkg/set"
) )
var ticker uint64 = 0
// GenerateFreshName generates a variable name that is not in the used set. // GenerateFreshName generates a variable name that is not in the used set.
// This function does not mutate the used set. // This function does not mutate the used set.
func GenerateFreshName(used set.Set[string]) string { func GenerateFreshName(used set.Set[string]) string {
for i := uint64(0); ; i++ { for i := uint64(0); ; i++ {
attempt := "_" + string(strconv.AppendUint(nil, ticker, 10)) attempt := "_" + string(strconv.AppendUint(nil, i, 10))
if !used.Has(attempt) { if !used.Has(attempt) {
return attempt return attempt