Reading from the storage multiple times

[code423n4]

Handle

pauliax

Vulnerability details

Impact

There are places where the same storage variable is accessed multiple times in the same function. It would be more gas efficient to cache these variables and re-use them where necessary. E.g. function tokenPrice accesses pools[_token].pairToken 3 times or function _unwrapUniPosition reads from uniPosition[_account] 3 times.

Recommended Mitigation Steps

Extract the value to a temporary variable and re-use it.

[talegift]

This looks more like a tip rather than a specific report. Would be great to have a full list of all occurances.

[ghoul-sol]

incomplete report, invalid