Skip to content

Conversation

@MathisGD
Copy link
Contributor

@MathisGD MathisGD commented Nov 19, 2025

fix verif after the merge with main

@MathisGD MathisGD self-assigned this Nov 19, 2025
chatgpt-codex-connector[bot]

This comment was marked as outdated.

Copy link
Collaborator

@QGarchery QGarchery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice !

uint256 expected = Utils.expectedSupplyAssets(e, MorphoMarketV1, marketParams, MorphoMarketV1Adapter);
uint256 allocation = MorphoMarketV1Adapter.allocation();
uint256 expected = Utils.expectedSupplyAssets(e, MorphoMarketV1, MorphoMarketV1Adapter.marketParams(), MorphoMarketV1Adapter.supplyShares());
require expected < 2 ^ 128;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can "prove" this property, because we have that supply shares are less than total supply shares (safe because in morpho-blue we proved that total supply shares is the sum of all supply shares), and we have that total supply assets is less than 2^128

@MathisGD MathisGD mentioned this pull request Nov 20, 2025
@MathisGD MathisGD merged commit 46b4876 into feat/one-market-per-adapter Nov 20, 2025
36 checks passed
@MathisGD MathisGD deleted the chore/merge-main branch November 20, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants