search

nervosnetwork / ckb

The Nervos CKB is a public permissionless blockchain, and the layer 1 of Nervos network.
https://www.nervos.org
MIT License
1.15k stars 228 forks source link

chore: minor refactor on tx pool #4464

Closed quake closed 4 months ago

quake commented 4 months ago

What problem does this PR solve?

this PR contains two minor refactor commits:

  1. refactor fill_proposals fn, removed the length checking in each iteration and make it easier to understand.
  2. refactor score_sorted_iter_by fn, removed the vec#contains for single status filtering.

Check List

Tests

Release note 

None: Exclude this PR from the release note.