Skip to content

Commit

Permalink
address PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
0xAlcibiades committed Dec 9, 2022
1 parent 28f98bb commit d83c2dd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/OptionSettlementEngine.sol
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ contract OptionSettlementEngine is ERC1155, IOptionSettlementEngine {
//////////////////////////////////////////////////////////////*/

/// @notice Accessor for Option contract details
mapping(uint160 => OptionRecord) internal optionRecords;
mapping(uint160 => OptionEngineState) internal optionRecords;

/*//////////////////////////////////////////////////////////////
// State variables - Public
Expand Down Expand Up @@ -310,7 +310,7 @@ contract OptionSettlementEngine is ERC1155, IOptionSettlementEngine {
uint256 encodedOptionId = uint256(optionKey) << OPTION_ID_PADDING;

// Get the option record and check that it's valid to write against
OptionRecord storage optionState = optionRecords[optionKey];
OptionEngineState storage optionState = optionRecords[optionKey];

// Make sure the option exists, and hasn't expired
uint40 expiry = optionState.option.expiryTimestamp;
Expand Down
2 changes: 1 addition & 1 deletion src/interfaces/IOptionSettlementEngine.sol
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ interface IOptionSettlementEngine {
mapping(uint16 => bool) doesBucketHaveUnexercisedOptions;
}

struct OptionRecord {
struct OptionEngineState {
/// @notice Information about the option type
Option option;
/// @notice Information about the option's claim buckets
Expand Down

0 comments on commit d83c2dd

Please sign in to comment.