replace time.After to time.NewTimer

[0xcb9ff9]

Description

time.After exist memory leak

Changes include

• Bugfix (non-breaking change that solves an issue)
• Hotfix (change that solves an urgent issue, and requires immediate attention)
• New feature (non-breaking change that adds functionality)
• Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

• I have assigned this PR to myself
• I have added at least 1 reviewer
• I have added the relevant labels
• I have updated the official documentation
• I have added sufficient documentation in code

Testing

• I have tested this code with the official test suite
• I have tested this code manually

Manual tests

create pprof file

$ go test -coverprofile coverage.out -timeout 28m -memprofile mem.pprof ./txpool

analysis pprof file

$ go tool pprof -http 0.0.0.0:8080 -no_browser mem.pprof

browser open http://0.0.0.0:8080

[codecov-commenter]

Codecov Report

Merging #181 (d887d5c) into dev (20f2db1) will increase coverage by 0.08%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##              dev     #181      +/-   ##
==========================================
+ Coverage   51.27%   51.35%   +0.08%     
==========================================
  Files         113      113              
  Lines       16875    16881       +6     
==========================================
+ Hits         8652     8670      +18     
+ Misses       7485     7474      -11     
+ Partials      738      737       -1     

Impacted Files	Coverage Δ
consensus/ibft/ibft.go	37.11% <20.00%> (-0.08%)	⬇️
protocol/sync_peer.go	70.00% <50.00%> (+1.00%)	⬆️
protocol/testing.go	84.03% <66.66%> (+0.13%)	⬆️
network/server.go	75.31% <100.00%> (+0.53%)	⬆️
network/server_discovery.go	80.89% <0.00%> (+7.00%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[DarianShawn]

Excellent job. 💯

A little bit change would make it better.