1. qs-badge-fixed In the repo `naym-coin`: some items are imported but not used. 2. qs-badge-fixed In `naym-coin/InitDiamond`: there is a "//todo" comment. 3. In `contracts-v3/ACLFacet`: consider caching in a local variable the value of `LibHelpers._stringToBytes32(_role)` to save gas. 4. qs-badge-fixed In `contracts-v3/LibSimplePolicy `: in `_payClaim()`, we could replace `>` with `>=` in the following statement: `require(availableBalance > _amount, "not enough balance")`. 5. In `contracts-v3/LibFeeRouter`:
a. In `_addFeeSchedule()`, consider caching in a local variable the value of `feeSchedule.receiver.length` to save gas.
b. Code could be reused between the functions `_payPremiumFees()`, `_payTradingFees()`, `_calculatePremiumFees()` and `_calculateTradingFees()` to reduce the code complexity.
c. qs-badge-fixed In `_calculatePremiumFees()`, the line `uint256 totalReceiverCount;` can be merged with the next one.
d. qs-badge-fixed The fields `FeeAllocation.from` and `FeeAllocation.token` are not used.
e. qs-badge-fixed In the function `_payPremiumFees()`, the fact that `fee > 0` is only checked in the second for loop. 6. qs-badge-fixed In `contracts-v3/SimplePolicyFacet`: in `getSimplePolicyInfo()`, consider caching in a local variable the value of `LibSimplePolicy._getSimplePolicyInfo(_policyId)` to save gas. 7. qs-badge-fixed In `contracts-v3/StakingFacet`: according to [Nayms code standard](https://docs.nayms.com/contracts-docs-copy/docs-static/code_standards/), `AppStorage` should not be accessed via facets. 8. qs-badge-fixed In `contracts-v3/LibTokenizedVaultIO`: double-check if the keyword `indexed` should be used for the events `ExternalDeposit` and `ExternalWithdraw`. 9. In `contracts-v3/LibMarket`:
a. qs-badge-fixed In `_createOffer()`, cache in a local variable the values of `s.objectMinimumSell[_buyToken]` and `s.objectMinimumSell[_sellToken]` to save gas.
b. qs-badge-fixed In `_takeOffer()`, cache in a local variable the values of `s.objectMinimumSell[s.offers[_offerId].sellToken]` to save gas.
c. Consider using a constant instead of a magic value `1`. 10. In `contracts-v3/LibTokenizedVaultStaking`:
a. qs-badge-fixed In `addUniqueValue()`, cache in a local variable the value of `rewards.currencies.length` to save gas.
b. qs-badge-fixed In `addUniqueValue()`, the line `rewards_.lastPaidInterval = i;` could be removed from the loop, and updated after the loop, based on the size of the list.
c. qs-badge-fixed Code could be reused between the functions `_getStakingStateWithRewardsBalances()` and `getStakingState()` to reduce the code complexity. 11. qs-badge-fixed In `contracts-v3/LibObject`: the function `LibObject._tokenSymbolNotUsed()` is not used. 12. qs-badge-fixed In `contracts-v3/Modifiers`: the modifier `assertIsInGroup`is not used anymore.