Skip to content

Conversation

@wit765
Copy link
Contributor

@wit765 wit765 commented Oct 27, 2025

This PR fixes function parameter filter shadowing method filter, enhancing code readability.

func (m *SortedMap) filter(filter func(*types.Transaction) bool) types.Transactions {

@wit765 wit765 requested a review from rjl493456442 as a code owner October 27, 2025 09:48
@wit765 wit765 changed the title core: rename parameter filter to filterFun for readability core: fix parameter filter shadowing method filterFun Oct 27, 2025
@fjl
Copy link
Contributor

fjl commented Oct 27, 2025

The code works as-written.

@fjl fjl closed this Oct 27, 2025
@wit765
Copy link
Contributor Author

wit765 commented Oct 27, 2025

Although this code works, but it is not a good coding style.

@wit765 wit765 deleted the rename-filter branch November 1, 2025 04:17
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