QA Report

[code423n4]

Non Critical

Use memory instead of storage, when you just read state vars:

It's more readable to use memory keyword, when we don't want to write changes or just read the state vars.

• https://github.com/code-423n4/2022-04-backed/blob/main/contracts/NFTLoanFacilitator.sol#L338
• https://github.com/code-423n4/2022-04-backed/blob/main/contracts/NFTLoanFacilitator.sol#L351
• https://github.com/code-423n4/2022-04-backed/blob/main/contracts/NFTLoanFacilitator.sol#L364

It's also more gas efficient but these are external view functions so gas doesn't matter here.

[wilsoncusack]

using memory unnecessary copies the whole struct into memory which is not needed. This is more gas