Fix RecentWasms cache bug by using pointers in methods

[KolbyML]

Resolves NIT-4128

pulls in OffchainLabs/go-ethereum#581

[github-actions]

❌ 2 Tests Failed:

Tests completed	Failed	Passed	Skipped
2173	2	2171	0

View the top 2 failed tests by shortest run time

TestMultiGasConstraintsStorage

Stack Traces | 0.000s run time

=== RUN   TestMultiGasConstraintsStorage
=== PAUSE TestMultiGasConstraintsStorage
=== CONT  TestMultiGasConstraintsStorage
    constraints_test.go:346: 
        	Error Trace:	/home/runner/work/nitro/nitro/precompiles/constraints_test.go:346
        	Error:      	Not equal: 
        	            	expected: 0x1
        	            	actual  : 0x3
        	Test:       	TestMultiGasConstraintsStorage
--- FAIL: TestMultiGasConstraintsStorage (0.00s)

TestTimeboostTxsTimeoutByBlock

Stack Traces | 33.350s run time

... [CONTENT TRUNCATED: Keeping last 20 lines]
DEBUG[11-26|16:23:48.248] Dereferenced trie from memory database   nodes=19  size=3.64KiB   time="36.027µs"  gcnodes=1021 gcsize=196.95KiB  gctime=2.479761ms  livenodes=0     livesize=0.00B
DEBUG[11-26|16:23:48.248] Dereferenced trie from memory database   nodes=0   size=0.00B     time=231ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.479871ms  livenodes=0     livesize=0.00B
DEBUG[11-26|16:23:48.248] Dereferenced trie from memory database   nodes=0   size=0.00B     time=181ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.479961ms  livenodes=0     livesize=0.00B
INFO [11-26|16:23:48.248] Blockchain stopped
DEBUG[11-26|16:23:48.251] redis producer: check responses starting
DEBUG[11-26|16:23:48.251] checkResponses                           responded=0 errored=0 checked=0
DEBUG[11-26|16:23:48.256] redis producer: check responses starting
DEBUG[11-26|16:23:48.298] checkResponses                           responded=0 errored=0 checked=0
DEBUG[11-26|16:23:48.298] Dereferenced trie from memory database   nodes=0   size=0.00B     time=530ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.480211ms  livenodes=0     livesize=0.00B
DEBUG[11-26|16:23:48.298] Dereferenced trie from memory database   nodes=0   size=0.00B     time=811ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.480581ms  livenodes=0     livesize=0.00B
DEBUG[11-26|16:23:48.298] Dereferenced trie from memory database   nodes=0   size=0.00B     time=421ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.480842ms  livenodes=0     livesize=0.00B
TRACE[11-26|16:23:48.298] Refreshing our lock                      id=auctioneer-79fa6b06-1764174195559988472
DEBUG[11-26|16:23:48.298] Dereferenced trie from memory database   nodes=0   size=0.00B     time=190ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.480912ms  livenodes=0     livesize=0.00B
DEBUG[11-26|16:23:48.298] Dereferenced trie from memory database   nodes=0   size=0.00B     time=471ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.481183ms  livenodes=0     livesize=0.00B
DEBUG[11-26|16:23:48.298] Dereferenced trie from memory database   nodes=0   size=0.00B     time=170ns       gcnodes=1021 gcsize=196.95KiB  gctime=2.481253ms  livenodes=0     livesize=0.00B
TRACE[11-26|16:23:48.298] P2P networking is spinning down
ERROR[11-26|16:23:48.298] Context closed, autonomous auctioneer shutting down
ERROR[11-26|16:23:48.298] Context closed, autonomous auctioneer shutting down
INFO [11-26|16:23:48.298] Context done while waiting redis streams to be ready, failed to start
--- FAIL: TestTimeboostTxsTimeoutByBlock (33.35s)

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov

[KolbyML]

@eljobe this PR is ready to merge