Skip to content

handle arbitrary slice types in sort builtin#968

Open
netliomax25-code wants to merge 1 commit into
expr-lang:masterfrom
netliomax25-code:sort-reflect-fallback
Open

handle arbitrary slice types in sort builtin#968
netliomax25-code wants to merge 1 commit into
expr-lang:masterfrom
netliomax25-code:sort-reflect-fallback

Conversation

@netliomax25-code
Copy link
Copy Markdown

sort only special-cases []any/[]int/[]float64/[]string, so a dynamically typed slice like []int64 or []uint falls through the switch and returns an empty slice with no error, silently dropping every element. Handle any slice or array through reflection like reverse and uniq do.

@antonmedv
Copy link
Copy Markdown
Member

looks like a lot of generated expressions now are falling

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants