Skip to content

core: use slices package for sorting#27489

Merged
fjl merged 6 commits into
ethereum:masterfrom
danlaine:use-slices-package-for-sorting-core
Jun 20, 2023
Merged

core: use slices package for sorting#27489
fjl merged 6 commits into
ethereum:masterfrom
danlaine:use-slices-package-for-sorting-core

Conversation

@danlaine
Copy link
Copy Markdown

See #27474

Comment thread core/mkalloc.go Outdated
Comment thread core/rawdb/accessors_chain.go Outdated
Comment thread core/state/snapshot/sort.go Outdated
Copy link
Copy Markdown
Member

@rjl493456442 rjl493456442 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few nitpicks, otherwise lgtm

@fjl fjl changed the title core; use slices package for sorting core: use slices package for sorting Jun 19, 2023
@fjl fjl force-pushed the use-slices-package-for-sorting-core branch from 803754b to 363b8d2 Compare June 19, 2023 21:47
@fjl fjl merged commit 154b016 into ethereum:master Jun 20, 2023
devopsbo3 pushed a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
Co-authored-by: Felix Lange <fjl@twurst.com>
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Oct 30, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Oct 30, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Nov 13, 2025
AnilChinchawale pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Nov 15, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Nov 28, 2025
AnilChinchawale pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Dec 7, 2025
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.

3 participants